[PR #3016] [MERGED] [client] Remove redundant function parameters in HandleEvent #4673

New issue

Closed

opened 2026-05-05 14:46:31 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-05 14:46:31 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/fatedier/frp/pull/3016
Author: @chenjiayao
Created: 7/14/2022
Status: ✅ Merged
Merged: 7/14/2022
Merged by: @fatedier

Base: dev ← Head: dev

📝 Commits (1)

634c741 [client] Remove redundant function parameters

📊 Changes

3 files changed (+4 additions, -11 deletions)

View changed files

📝 client/event/event.go (+1 -8)
📝 client/proxy/proxy_manager.go (+1 -1)
📝 client/proxy/proxy_wrapper.go (+2 -2)

📄 Description

The type parameter of the HandleEvent function seems to be useless, can we remove this parameter?

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/fatedier/frp/pull/3016 **Author:** [@chenjiayao](https://github.com/chenjiayao) **Created:** 7/14/2022 **Status:** ✅ Merged **Merged:** 7/14/2022 **Merged by:** [@fatedier](https://github.com/fatedier) **Base:** `dev` ← **Head:** `dev` --- ### 📝 Commits (1) - [`634c741`](https://github.com/fatedier/frp/commit/634c74161c835ac3b14bca564278ff77240901d9) [client] Remove redundant function parameters ### 📊 Changes **3 files changed** (+4 additions, -11 deletions) <details> <summary>View changed files</summary> 📝 `client/event/event.go` (+1 -8) 📝 `client/proxy/proxy_manager.go` (+1 -1) 📝 `client/proxy/proxy_wrapper.go` (+2 -2) </details> ### 📄 Description The type parameter of the HandleEvent function seems to be useless, can we remove this parameter? --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>