https://bugs.winehq.org/show_bug.cgi?id=7372
--- Comment #60 from Damjan Jovanovic damjan.jov@gmail.com --- (In reply to Pekka Helenius from comment #59)
The latest patch provided by Damjan Jovanovic works for Wine 1.7.29. Tested with Linux version of Firefox (native) and Powerpoint 2010 (wine installation). Should the patch be implemented to some of the next wine versions?
No, the patch was rejected. IRC discussion between me (as dacha) and Alexandre (as julliard) below:
Aug 28 18:55:03 <dacha> julliard can i please have some feedback on http://source.winehq.org/patches/data/105925 Aug 28 19:16:40 <julliard> dacha: once again it looks overly complicated Aug 28 19:17:59 <dacha> julliard: it actually needs to be vastly more complicated to be fully complete and correct, eg. to handle all possible placements of HTML comments. Aug 28 19:18:48 <dacha> julliard: why can't i just use libxml2's HTML parser like before? Aug 28 19:18:57 <julliard> dacha: well sure, but you are not going to be able to do the complex version if the simple version is already so messy Aug 28 19:21:57 <dacha> julliard: what exactly do you mean by messy? magic numbers or what? Aug 28 19:23:43 <julliard> magic numbers, parsing the text twice, copying strings multiple times, useless ERRs, etc. Aug 28 19:28:58 <dacha> julliard: ok thank you, i'll see what i can do. what about using libxml2? Aug 28 19:29:54 <julliard> dacha: using libxml2 has many other problems Aug 28 19:30:52 <dacha> julliard: but we use it in msxml3 don't we? Aug 28 19:31:30 <julliard> yes, but we don't want it all over the place Aug 28 19:31:41 <julliard> if you really wanted that you should load msxml3, but again we don't want that Aug 28 19:31:49 <julliard> simple string parsing is good enough for this Aug 28 19:35:18 <nsivov> dacha, for simplified parser take a look at ntdll/actctx.c, it works for known document structure, it's simple and small enough
Also we discovered that (at least) <A> and <IMG> tags can have C:\path\to\something style paths, which means the HTML needs to be deeply parsed and those paths converted to *nix paths. The reverse holds true when exporting HTML from Wine, and IIRC, doesn't happen yet either. So there's a lot of work remaining :-(. Anybody want to help?