Improved some conditions #1386

Merged

ReneWerner87 merged 16 commits intogofiber:masterfrom Ja7ad:master

Jun 17, 2021

+23-23

Commits on Jun 16, 2021

simplify u <= (1<<7)-1 to u < (1 << 7)
Javad
committed
It's not recommended to use len for empty string, we can check with string with ""
Javad
committed
It's not recommended to use len for empty string, we can check with string with ""
Javad
committed
It's not recommended to use len for empty string, we can check with string with ""
Javad
committed
It's not recommended to use len for empty string, we can check with string with ""
Javad
committed
Instead Bool comparison can using simplified bool check if !var = false checking
Javad
committed
Unnecessary use of fmt.Sprintf for value without format
Javad
committed
For check condition two value not required ! method
Javad
committed
nil check may not be enough for slice, better check with len
Javad
committed
function parameters combined
Javad
committed
When the form returns error information, the text content should not start with a capital letter or end with a punctuation mark
Javad
committed
error var invalidPath should have name of the form errFoo, It is recommended that the error variables that are part of an API should be named
Javad
committed
change to condition len(x), it's faster gofiber#1386 (comment)
Javad
committed

Commits on Jun 17, 2021