code review 10707045: misc/emacs: Add godef-jump-other-window

misc/emacs: Add godef-jump-other-window

This will behave like similar "*-other-window" functions in Emacs.

Default key bind is C-x 4 C-c C-j – while awkward, it follows both
the convention for other-window functions and the convention for
not using user- or emacs-reserved keys.

[Dominik Honnef, 2013-06-28 13:50:21]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://code.google.com/p/go

[Dominik Honnef, 2013-07-10 07:58:35]

Ping.

Related: I realize that changes to misc/emacs have low priority and are probably
somewhat "annoying". I have more changes pending, should I ball them up in a
single CL or keep splitting them into sensible units?

[adonovan, 2013-07-10 12:13:20]

LGTM

> Related: I realize that changes to misc/emacs have low priority and are
> probably somewhat "annoying". I have more changes pending, should I ball
> them up in a single CL or keep splitting them into sensible units?

I'm sorry I've let this change (and others) sit so long; I think I have a hard
time noticing e-mail that isn't addressed directly to me.  Don't be afraid to
put my email address (adonovan@google.com) directly in the Reviewer line of the
CL description.

cheers
alan

[Dominik Honnef, 2013-07-10 12:50:44]

On 2013/07/10 12:13:20, adonovan wrote:
> LGTM
> 
> > Related: I realize that changes to misc/emacs have low priority and are
> > probably somewhat "annoying". I have more changes pending, should I ball
> > them up in a single CL or keep splitting them into sensible units?
> 
> I'm sorry I've let this change (and others) sit so long; I think I have a hard
> time noticing e-mail that isn't addressed directly to me.  Don't be afraid to
> put my email address (mailto:adonovan@google.com) directly in the Reviewer
line of the
> CL description.
> 
> cheers
> alan

Okay, will do. I wasn't really sure how the Go team manages CLs nowadays because
of the new tools for handling them, ways of assigning them etc. I'll stick to
adding you directly to the reviewers then, thanks.

[Dominik Honnef, 2013-07-17 21:44:50]

Want to commit? Unfortunately I am not permitted to ;)

[adonovan, 2013-07-17 22:16:47]

*** Submitted as https://code.google.com/p/go/source/detail?r=e49eb0bc44bc ***

misc/emacs: Add godef-jump-other-window

This will behave like similar "*-other-window" functions in Emacs.

Default key bind is C-x 4 C-c C-j – while awkward, it follows both
the convention for other-window functions and the convention for
not using user- or emacs-reserved keys.

R=golang-dev, adonovan
CC=golang-dev
https://codereview.appspot.com/10707045

Committer: Alan Donovan <adonovan@google.com>