Re: the last patch that I sent you BREAKS isindex.
On Thu, Apr 17, 2008 at 06:17:04AM +0000, corvid wrote:
I shouldn't have moved isindex's connectActivate to close_form. If you wish, I can make a new version.
Yes, please.
By the way, I found that the top of open_input needs if (html->InFlags & IN_SELECT) { MSG_HTML("INPUT inside SELECT\n"); return; } Otherwise, if another input sneaks in there, close_select will crash.
Please include it in the patch. -- Cheers Jorge.-
Jorge wrote:
On Thu, Apr 17, 2008 at 06:17:04AM +0000, corvid wrote:
I shouldn't have moved isindex's connectActivate to close_form. If you wish, I can make a new version.
Yes, please.
I just wanted to let people on the list know that that patch wasn't sent to the list, so this isn't about anything that you might have applied to your tree.
By the way, I found that the top of open_input needs if (html->InFlags & IN_SELECT) { MSG_HTML("INPUT inside SELECT\n"); return; } Otherwise, if another input sneaks in there, close_select will crash.
Please include it in the patch.
I'm planning to make a patch that contains a few IN_* tests that are needed here and there...
participants (2)
-
corvid＠lavabit.com -
jcid＠dillo.org