undefined | Better HN

0 pointsjchw5y ago0 comments

The second one is pretty intentional - it'd be annoying it were downright impossible.

The first and third one fails on go vet. The first one also fails to compile if you never read from err in the entire function.

0 comments

Vet accepts the third, and may (I forget) accept the first if err is redeclared or simply assigned.

jchwOP5y ago

My bad, I was thinking of errcheck, which is more thorough and integrated into most Go linting tools like Go CI linter.

In any case, it’s trivial to detect via static analysis.

j / k navigate · click thread line to collapse

0 pointsjchw5y ago0 comments

The second one is pretty intentional - it'd be annoying it were downright impossible.

The first and third one fails on go vet. The first one also fails to compile if you never read from err in the entire function.

Vet accepts the third, and may (I forget) accept the first if err is redeclared or simply assigned.

jchwOP5y ago

My bad, I was thinking of errcheck, which is more thorough and integrated into most Go linting tools like Go CI linter.

In any case, it’s trivial to detect via static analysis.

j / k navigate · click thread line to collapse