Jerry James writes: > x_event_copy = &EVENT_MAGIC_X_EVENT (emacs_event); [...] > SET_EVENT_MAGIC_X_EVENT (emacs_event, *x_event_copy); [...] > As far as I can see, the SET_EVENT_MAGIC_X_EVENT call is a no-op. I'd > like to remove it. Any objections? I agree, especially in the face of valgrind's complaint. Nuking it is OK by me.