[Bug 7372] Can't copy and paste between native apps (e.g., Open Office, Firefox, Gimp) and some Wine apps (affects Dreamweaver, Photoshop, Powerpoint, Excel)

[wine-bugs at winehq.org]

https://bugs.winehq.org/show_bug.cgi?id=7372

--- Comment #60 from Damjan Jovanovic <damjan.jov at 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?

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.