Version 8 of tab+frame patch: smallproblemwithwhitespaceinmultilength
Hi'all, I just noticed that the current version of my tab+frames patch does not handle multilengths with whitespace correctly. I already fixed it in my version, will include the fix in the next version of the patch. If you encounter framesets which look weird, it may be caused by this problem. As the multilength parser was based on the one found in html.c, this problem is also present in anything which uses Html_parse_multi_length_list(), or which feeds length strings with leading whitespace to Html_parse_length_or_multi_length(). So, a multilength like "*,*" works fine, as does "* ,*", but "*. *" does not work correctly. As far as I can see, these functions are used in tables and embedded widgets. My fix in gtkframeset.c was to rewrite the parser into something slightly more robust (using g_strsplit and g_strstrip). Something similar is needed for html.c. I might include this in my patch, but unpatched Dillo also needs a fix so it is better solved in CVS. Cheers//Frank -- WWWWW ________________________ ## o o\ / Frank de Lange \ }# \| / +46-734352015 \ \ `--| _/ <Hacker for Hire> \ `---' \ +31-640037120 / \ frank@unternet.org / `------------------------' [ "Omnis enim res, quae dando non deficit, dum habetur et non datur, nondum habetur, quomodo habenda est." ]
participants (1)
-
Frank de Lange