Re: local keymap patch for key-binding

[Richard Stallman]

Why add the argument LOCATION?  Is it just for the sake of
command-rebinding?

Why the need to switch buffers?

Aside from that, some little details:

    +   * keymap.c: include "window.h".

You need to add a dependency in Makefile.in for that.

    !       if (SYMBOLP (AREF(location,0)) && ASIZE(location) > 1)
    !   event = AREF(location,1);
    !       else
    !   event = AREF(location,0);

To follow our coding style you need additional spaces here:

    !       if (SYMBOLP (AREF (location,0)) && ASIZE (location) > 1)
    !   event = AREF (location, 1);
    !       else
    !   event = AREF (location, 0);

A comma should always be followed by a space.
Outside of a parenthesis there must be a space,
unless the same spot is inside another parenthesis
or followed by a semicolon.

There needs to be documentation for LOCATION in etc/NEWS and the Lisp
Manual.

    !     /* Key sequences beginning with mouse clicks are
    !        read using the keymaps in the buffer clicked on,
    !        not the current buffer.  If we're at the
    !        beginning of a key sequence, switch buffers.  */

I can't understand the second sentence, "If we're at the beginning of
a key sequence".  Since key-binding always operates on a key sequence,
and since we're not in the middle of a loop which scans that key
sequence, what does it mean to say that we are, or are not, "at the
beginning" of it?