passing the input using stdin without the filename. I don't think breaking
goimports is worth it (specially, since it's intended as a gofmt
replacement and the latter also works with just stdin, no filename), but
that's not my call to make. Also, it might be desirable to keep
tools/imports (where +90% of the execution time of goimports is spent) able
to work only on memory, with no filesystem available.
In any case, teaching goimports about other identifiers in the same package
feels like a bandaid solution to me. It would be better to just make it
fast at finding Go packages and the other issues would just go away (if you
take finding Go pkgs out of the equation, goimports takes less than a few
milliseconds to do its job).
On Friday, May 15, 2015 at 12:00:48 PM UTC+1, rog wrote:
On 14 May 2015 at 19:44, Alberto García Hierro <[email protected]
<javascript:>> wrote:
providing standard input, from which it can infer the correct package
to parse.
--On 14 May 2015 at 19:44, Alberto García Hierro <[email protected]
<javascript:>> wrote:
That seems like it could cause subtle problems. What happens if I'm editing
foo/bar/baz.go? The tool will read the definitions from package foo (since
it's the working directory where I invoked $EDITOR) while it's actually
feeding it a file from package bar.
godef currently requires that you pass it the file name as well asfoo/bar/baz.go? The tool will read the definitions from package foo (since
it's the working directory where I invoked $EDITOR) while it's actually
feeding it a file from package bar.
providing standard input, from which it can infer the correct package
to parse.
You received this message because you are subscribed to the Google Groups "golang-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [email protected].
For more options, visit https://groups.google.com/d/optout.
