(These are generic comments, I haven't really looked at the patch.) FWIW my feelings are that having XEmacs internal stuff in (supposedly) externally maintained files (browse-url.el) is less than perfect. (Synching the file with the external source becomes a pain, someone need to look over the XEmacs specific and decide that they indeed are XEmacs specific and that they are still valid even after the other changes. This review has to be done every time the file is synched. There are more fun things to do..) But I wouldn't have anything against putting this in a new file, e.g. xemacs-url.el, in the mail-lib package, since I agree it may be easier to maintain this in a package than in the core. So assuming the patch is good (I didn't look at it, sorry) and that it doesn't _have_ to be in browse-url.el but just in any package (and I guess mail-lib would be a suitable place), MHO is to move it into a mail-lib/xemacs-url.el.