Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove redundant parentheses and update comments #2082

Merged
merged 27 commits into from
Sep 11, 2022
Merged
Changes from all commits
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
fe7f105
fix unhandled errors
Kamandlou Aug 26, 2022
7f1028d
fix unhandled error in cache package test
Kamandlou Aug 26, 2022
bf387fa
omit variable type
Kamandlou Aug 26, 2022
d09213b
omit variable type
Kamandlou Aug 26, 2022
14b6646
rename variable because collide with the imported package name
Kamandlou Aug 26, 2022
1c45f7e
handle file error on closing
Kamandlou Aug 26, 2022
a021d0c
Merge branch 'gofiber:master' into master
Kamandlou Aug 26, 2022
89bf878
fix unhandled in common_linux.go
Kamandlou Aug 28, 2022
8dded84
Merge branch 'gofiber:master' into master
Kamandlou Aug 29, 2022
08a9256
fix unhandled errors in helpers_test.go
Kamandlou Aug 29, 2022
038315d
fix unhandled errors in listen_test.go
Kamandlou Aug 30, 2022
e5b8dad
remove unused parameter in emptyHandler method
Kamandlou Aug 30, 2022
92e6920
refactor path.go
Kamandlou Sep 1, 2022
3086fd2
unhandled error in hooks test
Kamandlou Sep 1, 2022
73c9987
Merge branch 'gofiber:master' into master
Kamandlou Sep 2, 2022
8b466c3
fix unhandled errors in app_test.go
Kamandlou Sep 2, 2022
f1f46b1
Merge pull request #1 from Kamandlou/dev
Kamandlou Sep 2, 2022
a125a5f
fix unhandled errors in ctx_test.go
Kamandlou Sep 2, 2022
d00514c
Merge pull request #2 from Kamandlou/dev
Kamandlou Sep 3, 2022
81a0bc5
✨ fix unhandled errors in helpers_test.go
Kamandlou Sep 4, 2022
9ba9001
Merge branch 'gofiber:master' into master
Kamandlou Sep 4, 2022
83885d6
Merge pull request #3 from Kamandlou/dev
Kamandlou Sep 4, 2022
cec4b83
revert app_test.go
Kamandlou Sep 5, 2022
02aa986
Merge pull request #4 from Kamandlou/dev
Kamandlou Sep 5, 2022
afd7b13
remove redundant parentheses and update comments
Kamandlou Sep 11, 2022
9bbd11b
Merge branch 'gofiber:master' into master
Kamandlou Sep 11, 2022
8f7815b
Merge pull request #5 from Kamandlou/dev
Kamandlou Sep 11, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions ctx.go
Original file line number Diff line number Diff line change
Expand Up @@ -464,7 +464,7 @@ func (c *Ctx) Cookie(cookie *Cookie) {
fasthttp.ReleaseCookie(fcookie)
}

// Cookies is used for getting a cookie value by key.
// Cookies are used for getting a cookie value by key.
// Defaults to the empty string "" if the cookie doesn't exist.
// If a default value is given, it will return that value if the cookie doesn't exist.
// The returned value is only valid within the handler. Do not store any references.
Expand Down Expand Up @@ -933,7 +933,7 @@ func (c *Ctx) Params(key string, defaultValue ...string) string {
return defaultString("", defaultValue)
}

// Params is used to get all route parameters.
// AllParams Params is used to get all route parameters.
// Using Params method to get params.
func (c *Ctx) AllParams() map[string]string {
params := make(map[string]string, len(c.route.Params))
Expand Down Expand Up @@ -1253,7 +1253,7 @@ func (c *Ctx) getLocationFromRoute(route Route, params Map) (string, error) {

for key, val := range params {
isSame := key == segment.ParamName || (!c.app.config.CaseSensitive && utils.EqualFold(key, segment.ParamName))
isGreedy := (segment.IsGreedy && len(key) == 1 && isInCharset(key[0], greedyParameters))
isGreedy := segment.IsGreedy && len(key) == 1 && isInCharset(key[0], greedyParameters)
if isSame || isGreedy {
_, err := buf.WriteString(utils.ToString(val))
if err != nil {
Expand Down