cmd/vet: %w in Printf call should be flagged

[josharian]

package main

import (
	"errors"
	"fmt"
)

func main() {
	err := errors.New("ABC")
	fmt.Printf("error: %w\n", err)
}

https://play.golang.org/p/-kQ6HtIIMG7

vet should complain about the %w verb. Only in Errorf format strings does it have meaning.

[bcmills]

CC @ianthehat @matloob @dominikh

[mvdan]

This also seems like a pretty simple issue for anyone wanting to get started contributing to vet.

[Inconnu08]

I would love to take a shot on this. Any pointers to get started on this issue?

[agnivade]

The file in question is this one - https://github.com/golang/tools/blob/master/go/analysis/passes/printf/printf.go. Look at how other verbs are handled and take it from there.

[Inconnu08]

Ah I'm on the right track then. Thank you! I'm working on this 😃

[Inconnu08]

vet already complains about the %w verb in printf. I ran the above code and this is what I got:
./main.go:10:2: Printf format %w has unknown verb w.

[agnivade]

You are right. This was fixed by golang/tools@1d17272. The original issue was reported by @josharian himself which was about passing strings to a %w parameter. But the commit reported usages of %w with non-Errrof functions too.

The error message is Printf call has error-wrapping directive %w though, not the one you mentioned.

I will go ahead and close this one. Thanks for looking into it @Inconnu08.

[Inconnu08]

Exactly. But the end error message is the above one reported by vet. Anytime!

[josharian]

Oops, sorry. :( I thought I tested with tip but apparently only did 1.13. Thanks, @Inconnu08.