Skip to content

Commit

Permalink
refactor: Use stackBufSize as a global vars instead of a local vars
Browse files Browse the repository at this point in the history
  • Loading branch information
LimJiAn committed Sep 4, 2023
1 parent dc9cff7 commit 4aa07a2
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 2 additions & 1 deletion path.go
Expand Up @@ -5,6 +5,8 @@

package gin

const stackBufSize = 128

// cleanPath is the URL version of path.Clean, it returns a canonical URL path
// for p, eliminating . and .. elements.
//
Expand All @@ -19,7 +21,6 @@ package gin
//
// If the result of this process is an empty string, "/" is returned.
func cleanPath(p string) string {
const stackBufSize = 128
// Turn empty string into "/"
if p == "" {
return "/"
Expand Down
2 changes: 0 additions & 2 deletions tree.go
Expand Up @@ -657,8 +657,6 @@ walk: // Outer loop for walking the tree
// It returns the case-corrected path and a bool indicating whether the lookup
// was successful.
func (n *node) findCaseInsensitivePath(path string, fixTrailingSlash bool) ([]byte, bool) {
const stackBufSize = 128

// Use a static sized buffer on the stack in the common case.
// If the path is too long, allocate a buffer on the heap instead.
buf := make([]byte, 0, stackBufSize)
Expand Down

0 comments on commit 4aa07a2

Please sign in to comment.