http://bugs.winehq.com/show_bug.cgi?id=1073
------- Additional Comments From z_god@wanadoo.nl 2003-25-06 14:50 ------- Bug comments restored from Gmane.org:
When trying to Install StarCraft with latest CVS and earlier releases the StarCraft Installer shows only My Computer, no + to expand it and no Drives/Directories, only way to choose where to install it is to type the path to where you want it installed. I will try to do a regression test tomorrow (10/16/02) but Im not sure how tired I will be and how long it will take (slow 56k). I will post results of last working version when I can.
------- Additional Comments From Speeddymon <at> yahoo.com 2002-10-15 22:05 ------- Created an attachment (id=314) --> (http://bugs.winehq.com/attachment.cgi?id=314&action=view) Debug output of +listview, +treeview
------- Additional Comments From Speeddymon <at> yahoo.com 2002-10-16 13:54 ------- Reassigning to Dimi but keeping Mr. Bugs on the CC list.
------- Additional Comments From Speeddymon <at> yahoo.com 2002-10-27 13:24 ------- The patch causing this regression is:
http://www.winehq.com/hypermail/wine-cvs/2002/03/0070.html
If anyone wants a screenshot of it the way it used to look (before this patch) and the way it looks now (after this patch) let me know and I will post them.
Shoot I will post them anyways, in just a minute.
------- Additional Comments From Speeddymon <at> yahoo.com 2002-10-27 14:36 ------- Created an attachment (id=320) --> (http://bugs.winehq.com/attachment.cgi?id=320&action=view) The way the dialog looks after the patch
------- Additional Comments From Speeddymon <at> yahoo.com 2002-10-27 15:06 ------- Created an attachment (id=321) --> (http://bugs.winehq.com/attachment.cgi?id=321&action=view) ... and before
------- Additional Comments From Speeddymon <at> yahoo.com 2002-10-27 15:10 ------- Created an attachment (id=322) --> (http://bugs.winehq.com/attachment.cgi?id=322&action=view) DEbug output of +listview, +treeview (BEFORE PATCH)
------- Additional Comments From Speeddymon <at> yahoo.com 2002-10-27 19:05 ------- on the first debug log (the one for after the patch is committed) please ignore the debugger message, unless you want me to get a backtrace too?
------- Additional Comments From Speeddymon <at> yahoo.com 2002-10-29 15:41 ------- Dimi sent a patch to (hopefully) correct this,I will test it tonight and post the results then or tomorrow, whichever...
------- Additional Comments From Speeddymon <at> yahoo.com 2002-10-31 08:41 ------- tested patch, it didnt resolve the problem, but it was a step closer. he submitted another patch but i havent had a chance to test it out yet.. i will do that tonight and post the results. both patches have been committed to CVS already...
------- Additional Comments From Speeddymon <at> yahoo.com 2002-10-31 23:58 ------- ok 20021031 was released, and i tested the new patch, it didnt work either. he then priv emailed me a one-liner that worked. I am posting it here after the NEW debug output
------- Additional Comments From Speeddymon <at> yahoo.com 2002-11-01 00:02 ------- Created an attachment (id=325) --> (http://bugs.winehq.com/attachment.cgi?id=325&action=view) Hacky patch that works in case it doesn't get committed, this may break apps that send messages in unicode instead of ansi
------- Additional Comments From Speeddymon <at> yahoo.com 2002-11-01 00:03 ------- Created an attachment (id=326) --> (http://bugs.winehq.com/attachment.cgi?id=326&action=view) Debug log after the hacky patch is committed.
------- Additional Comments From Speeddymon <at> yahoo.com 2002-11-01 11:32 ------- Dimi had me add a couple of trace lines to the treeview code after the hack patch. For some weird reason the installer is expecting unicode even though it doesn't support it. He then asked me to add the patch to the code before Guy's patch and when I went to do that I found out that Guy's patch is the one that actually implements TREEVIEW_NotifyFormat so there was no way to do that.
Clip from email:
-begin- --- "Dimitrie O. Paun" <dpaun <at> rogers.com> wrote:
On November 1, 2002 02:00 am, you wrote:
after banging my head on my keyboard for a few minutes because i couldnt find treeview_notifyformat i went and looked at cvsweb and found out...there is no treeview_notifyformat before guy's patch so there is no way for me to get that kind of a debug output...
Good find. Please add this to Bugzilla. But it presents a big problem: if we behave correctly, we break the app. What do we do?
I'm wondering if maybe its my app thats broken already and thats why its sending the unicode message.. -end-