#30 Update or drop xterm patch and related code
joe ships a patch against xterm-200.
The said version of xterm was released in early 2005, i.e. 19 years ago. Maybe the patch did apply against a few subsequent xterm releases, but sure not for too long. I'm fairly certain that no one uses this patch anymore.
Some of the features made it into xterm or have an alternate solution. Some are probably not worth the effort of carrying. Some might be worth the effort of updating (but seriously, how many people out there would choose to use this particular terminal emulator and bother patching it? again I think it's damn close to zero).
I'm afraid the patch has its toll on the actual behavior of the software, as used by quite a few people out there (in xterm or in some other terminal emulator).
The one I've found right now is regarding the "outside of frame mouse coordinates" feature. Even in a patched xterm you have to enable this feature by emitting an escape sequence. Yet, mcoord() unconditionally expects this behavior and treats coordinate byte values 241..255 as negative coordinates rather than row/column 209..223.
I think it's high time to let go of these and clean up the code accordingly.