squirrel icon indicating copy to clipboard operation
squirrel copied to clipboard
verified publisher icon Masterminds

Redundant check in placeholder

Open levanidev opened this issue 3 years ago • 0 comments

https://github.com/Masterminds/squirrel/blob/9b18b54aed2aa831665b8e896cde063d51d921fa/placeholder.go#L97-L103

At line 100. since we already know that len(sql[p:]) > 1 from the previous if check, we know that len(sql[p:]) == 1 will always be false.

unless I'm missing something?

levanidev avatar Jun 01 '22 16:06 levanidev