Skip to content

chore: remove 3 lines of unused code in pkg/protocol/request.go#1502

Open
xiaosau wants to merge 1 commit into
cloudwego:mainfrom
xiaosau:dev/fixmp
Open

chore: remove 3 lines of unused code in pkg/protocol/request.go#1502
xiaosau wants to merge 1 commit into
cloudwego:mainfrom
xiaosau:dev/fixmp

Conversation

@xiaosau
Copy link
Copy Markdown

@xiaosau xiaosau commented May 23, 2026

Change-Id: Id7bf1485b0aefa27826043ab90e9558ec6feffa6

What type of PR is this?

chore: remove 3 lines of unused code in pkg/protocol/request.go

(Optional) Translate the PR title into Chinese.

(Optional) More detailed description for this PR(en: English/zh: Chinese).

zh(optional): 删除pkg/protocol/request.go下三行无用代码:
Go 标准库的 multipart.ReadForm 即使没有 file part 也会返回 File: map[string][]*FileHeader{} (非 nil map),即使返回nil map,后续直接读也会返回nil

// 标准库实现
func (r *Reader) readForm(maxMemory int64) (_ *Form, err error) {
	form := &Form{make(map[string][]string), make(map[string][]*FileHeader)}
        / .....
}

(Optional) Which issue(s) this PR fixes:

(Optional) The PR that updates user documentation:

Change-Id: Id7bf1485b0aefa27826043ab90e9558ec6feffa6
@xiaosau xiaosau requested review from a team as code owners May 23, 2026 08:17
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 23, 2026

CLA assistant check
All committers have signed the CLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants