http://bugs.winehq.org/show_bug.cgi?id=25463
--- Comment #19 from Jacek Caban jacek@codeweavers.com 2011-01-11 12:11:54 CST --- (In reply to comment #18)
Created an attachment (id=32809)
--> (http://bugs.winehq.org/attachment.cgi?id=32809) [details]
Update MK protocol handling to IInternetProtocolEx [v3]
(In reply to comment #17)
Thanks for the patch. I really hope we can avoid old-style UrlUnescape call here. I was expecting IUri APIs to allow us that, but I did some tests and it seems like it won't help here. Perhaps itss is responsible for unescaping? ...
I don't believe so, I through together a hacky test when I first started looking into this issue. To do that I hand-coding a call to open a valid CHM (with spaces in the filename) with IParseDisplayName_ParseDisplayName.
Good to know you already did that. Sadly it means that we really need unescaping in mk protocol handler.
I'm not sure if there's a good way to make a real test for this, but when I tested that hack I did not get a moniker back.
We could just escape URL in any way.
Also, attached is a patch for updating just IInternetProtocolEx, please see if this sufficiently addresses your concerns.
It looks good now. Please send the patch to wine-patches.