> As of [0], the Xorg man page for XFree reads: > > "If data is NULL, no operation is performed." > > It was changed to reflect the behavior that Xorg has always had. > XFree() has been #defined to free() since their initial code import in > 2003. However, it looks like other documentation was not updated, and > it's understandable if you want to keep the NULL checks. > > [0]: https://gitlab.freedesktop.org/xorg/lib/libx11/commit/78851f6
Agreed. > > I think the current sendevent() code is more readable. Agreed too. Please provide both changes as separate commits, one for XFree() unconditional call, and another one for refactoring sendevent(). Thanks!