On 02/11/2018 19:55, Karl Fogel wrote:
> 
> It's always a good idea to test changes in a freshly-built and freshly-started Emacs.  In this case, because there was no ";;;###autoload" marker before `bookmark-jump-other-frame', invoking that function in a fresh Emacs would generate an error.
Hi,

Oops i used to just work with eval-buffer on the lisp i needed, and i 
can see now this is not the right workflow for testing a patch.

So now i rather build Emacs to test patch properly, and obviously as you 
wrote adding the autoload magic comment fixed the issue. So here is the 
patch with this.

Regards
Pierre-Yves