There's a build error when you try to compile the app:

 ✗ go get github.com/3rf/go-unused-funcs
# github.com/3rf/go-unused-funcs/unused
/goprojects/src/github.com/3rf/go-unused-funcs/unused/unused_func_finder.go:91: cannot use serialRes.Callgraph (type []"golang.org/x/tools/oracle/serial".CallGraph) as type []"code.google.com/p/go.tools/oracle/serial".CallGraph in assignment

Many static analysis tools have ways to explicitly say "don't bother with this" in the code. Is there any way in my code to say "ignore this function please, codecoroner"?

Thanks!

Unused variables defined in the signature of a function should not be flagged as unused because those variables names are helpful as documentation even if not referenced in the function body.

The README doesn't give me any information on why I might choose this tool versus github.com/remyoudompheng/go-misc/deadcode from https://github.com/remyoudompheng/go-misc.

See build #45.1 for Go 1.5

When using codecoroner it would be useful to use exit code to detect any errors instead of parsing stdout/stderr.

Suggesting to add a non zero exit code if codecoroner found any unused objects.

Hopefully I will get some free time to update this before the end of May 2015. If you're feeling squirrelly, you can get it to compile by checking out an old version of the oracle package.

type set f(str string)

str should not be reported as unused as str is useful as documentation.

Note: this is a particular subcase of #21.

go get github.com/3rf/codecoroner

github.com/3rf/codecoroner/unused

work/src/github.com/3rf/codecoroner/unused/idents.go:56: impossible type switch case: kind (type "go/types".Object) cannot have dynamic type *"golang.org/x/tools/go/types".Func (wrong type for Parent method)
have Parent() *"golang.org/x/tools/go/types".Scope
want Parent() *"go/types".Scope
work/src/github.com/3rf/codecoroner/unused/idents.go:79: impossible type switch case: kind (type "go/types".Object) cannot have dynamic type *"golang.org/x/tools/go/types".Func (wrong type for Parent method)
have Parent() *"golang.org/x/tools/go/types".Scope
want Parent() *"go/types".Scope

Since this pkg relies heavily on Google packages with little external usage, it would be nice to vendor things to prevent disasters like the removal of callgraph analysis from the Oracle tool.

Code reachable by tests probably shouldn't be ignored.

For style consistency a method receiver variable might be defined for all methods of a type even for methods where the receiver is not used.
Signaling unused method receivers variables pollutes the output.

Related: #21

It'd be great if we can use that tool to then delete those unused functions

We should leverage the token.Position type to consistently print declaration location for both funcs and idents