http://bugs.winehq.org/show_bug.cgi?id=22287
Summary: Personal Chess Trainer v3.02, Run-time error '13': Type mismatch , regression Product: Wine Version: 1.1.42 Platform: x86 OS/Version: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: oleaut32 AssignedTo: wine-bugs@winehq.org ReportedBy: jskartman@gmail.com CC: nerv@dawncrow.de
Hi,
This is a regression, Personal Chess Trainer demo version 3.02 (actual demo version is called Chessimo v3.35 and needs .NET 3.5, so doesn't work), was working fine on wine 1.1.39 , but fails on 1.1.40. When I move a piece, the application shows a box with: Run-time error '13': Type mismatch
Doing a regression test shows:
71f24431b987d70053063a9f206913e7fff53edc is the first bad commit commit 71f24431b987d70053063a9f206913e7fff53edc Author: André Hentschel nerv@dawncrow.de Date: Wed Mar 3 19:57:27 2010 +0100
oleaut32: Fix localized date parsing.
:040000 040000 5cebc4bc57ab8449aa70bf9a0ca45cc31e52affb f740a9f92b8811f217b0e2365a804477acf427ba M dlls
Reverting the patch fails. I put oleaut32 component because the commit references to it.
http://bugs.winehq.org/show_bug.cgi?id=22287
Jeff Zaroyko jeffz@jeffz.name changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression Summary|Personal Chess Trainer |Personal Chess Trainer |v3.02, Run-time error '13': |v3.02, Run-time error '13': |Type mismatch , regression |Type mismatch
http://bugs.winehq.org/show_bug.cgi?id=22287
--- Comment #1 from Jaime jskartman@gmail.com 2010-04-06 07:08:09 --- reverting the patch mnually works!!
case 0x3: /* TTT TTTDD TTTDDD */ /* REVERT PATCH if (dp.dwCount > 4 && ((dp.dwFlags[3] & (DP_AM|DP_PM)) || (dp.dwFlags[4] & (DP_AM|DP_PM)) || (dp.dwFlags[5] & (DP_AM|DP_PM)))) hRet = DISP_E_TYPEMISMATCH; else */ if (dp.dwCount != 3 && dp.dwCount != 5 && dp.dwCount != 6) hRet = DISP_E_TYPEMISMATCH; st.wHour = dp.dwValues[0]; st.wMinute = dp.dwValues[1]; st.wSecond = dp.dwValues[2]; dwOffset = 3; dp.dwCount -= 3; break;
--- Comment #2 from Jaime jskartman@gmail.com 2010-04-06 07:08:10 --- reverting the patch mnually works!!
case 0x3: /* TTT TTTDD TTTDDD */ /* REVERT PATCH if (dp.dwCount > 4 && ((dp.dwFlags[3] & (DP_AM|DP_PM)) || (dp.dwFlags[4] & (DP_AM|DP_PM)) || (dp.dwFlags[5] & (DP_AM|DP_PM)))) hRet = DISP_E_TYPEMISMATCH; else */ if (dp.dwCount != 3 && dp.dwCount != 5 && dp.dwCount != 6) hRet = DISP_E_TYPEMISMATCH; st.wHour = dp.dwValues[0]; st.wMinute = dp.dwValues[1]; st.wSecond = dp.dwValues[2]; dwOffset = 3; dp.dwCount -= 3; break;
--- Comment #3 from Jaime jskartman@gmail.com 2010-04-06 07:08:11 --- reverting the patch mnually works!!
case 0x3: /* TTT TTTDD TTTDDD */ /* REVERT PATCH if (dp.dwCount > 4 && ((dp.dwFlags[3] & (DP_AM|DP_PM)) || (dp.dwFlags[4] & (DP_AM|DP_PM)) || (dp.dwFlags[5] & (DP_AM|DP_PM)))) hRet = DISP_E_TYPEMISMATCH; else */ if (dp.dwCount != 3 && dp.dwCount != 5 && dp.dwCount != 6) hRet = DISP_E_TYPEMISMATCH; st.wHour = dp.dwValues[0]; st.wMinute = dp.dwValues[1]; st.wSecond = dp.dwValues[2]; dwOffset = 3; dp.dwCount -= 3; break;
http://bugs.winehq.org/show_bug.cgi?id=22287
--- Comment #1 from Jaime jskartman@gmail.com 2010-04-06 07:08:09 --- reverting the patch mnually works!!
case 0x3: /* TTT TTTDD TTTDDD */ /* REVERT PATCH if (dp.dwCount > 4 && ((dp.dwFlags[3] & (DP_AM|DP_PM)) || (dp.dwFlags[4] & (DP_AM|DP_PM)) || (dp.dwFlags[5] & (DP_AM|DP_PM)))) hRet = DISP_E_TYPEMISMATCH; else */ if (dp.dwCount != 3 && dp.dwCount != 5 && dp.dwCount != 6) hRet = DISP_E_TYPEMISMATCH; st.wHour = dp.dwValues[0]; st.wMinute = dp.dwValues[1]; st.wSecond = dp.dwValues[2]; dwOffset = 3; dp.dwCount -= 3; break;
--- Comment #2 from Jaime jskartman@gmail.com 2010-04-06 07:08:10 --- reverting the patch mnually works!!
case 0x3: /* TTT TTTDD TTTDDD */ /* REVERT PATCH if (dp.dwCount > 4 && ((dp.dwFlags[3] & (DP_AM|DP_PM)) || (dp.dwFlags[4] & (DP_AM|DP_PM)) || (dp.dwFlags[5] & (DP_AM|DP_PM)))) hRet = DISP_E_TYPEMISMATCH; else */ if (dp.dwCount != 3 && dp.dwCount != 5 && dp.dwCount != 6) hRet = DISP_E_TYPEMISMATCH; st.wHour = dp.dwValues[0]; st.wMinute = dp.dwValues[1]; st.wSecond = dp.dwValues[2]; dwOffset = 3; dp.dwCount -= 3; break;
--- Comment #3 from Jaime jskartman@gmail.com 2010-04-06 07:08:11 --- reverting the patch mnually works!!
case 0x3: /* TTT TTTDD TTTDDD */ /* REVERT PATCH if (dp.dwCount > 4 && ((dp.dwFlags[3] & (DP_AM|DP_PM)) || (dp.dwFlags[4] & (DP_AM|DP_PM)) || (dp.dwFlags[5] & (DP_AM|DP_PM)))) hRet = DISP_E_TYPEMISMATCH; else */ if (dp.dwCount != 3 && dp.dwCount != 5 && dp.dwCount != 6) hRet = DISP_E_TYPEMISMATCH; st.wHour = dp.dwValues[0]; st.wMinute = dp.dwValues[1]; st.wSecond = dp.dwValues[2]; dwOffset = 3; dp.dwCount -= 3; break;
http://bugs.winehq.org/show_bug.cgi?id=22287
--- Comment #1 from Jaime jskartman@gmail.com 2010-04-06 07:08:09 --- reverting the patch mnually works!!
case 0x3: /* TTT TTTDD TTTDDD */ /* REVERT PATCH if (dp.dwCount > 4 && ((dp.dwFlags[3] & (DP_AM|DP_PM)) || (dp.dwFlags[4] & (DP_AM|DP_PM)) || (dp.dwFlags[5] & (DP_AM|DP_PM)))) hRet = DISP_E_TYPEMISMATCH; else */ if (dp.dwCount != 3 && dp.dwCount != 5 && dp.dwCount != 6) hRet = DISP_E_TYPEMISMATCH; st.wHour = dp.dwValues[0]; st.wMinute = dp.dwValues[1]; st.wSecond = dp.dwValues[2]; dwOffset = 3; dp.dwCount -= 3; break;
--- Comment #2 from Jaime jskartman@gmail.com 2010-04-06 07:08:10 --- reverting the patch mnually works!!
case 0x3: /* TTT TTTDD TTTDDD */ /* REVERT PATCH if (dp.dwCount > 4 && ((dp.dwFlags[3] & (DP_AM|DP_PM)) || (dp.dwFlags[4] & (DP_AM|DP_PM)) || (dp.dwFlags[5] & (DP_AM|DP_PM)))) hRet = DISP_E_TYPEMISMATCH; else */ if (dp.dwCount != 3 && dp.dwCount != 5 && dp.dwCount != 6) hRet = DISP_E_TYPEMISMATCH; st.wHour = dp.dwValues[0]; st.wMinute = dp.dwValues[1]; st.wSecond = dp.dwValues[2]; dwOffset = 3; dp.dwCount -= 3; break;
--- Comment #3 from Jaime jskartman@gmail.com 2010-04-06 07:08:11 --- reverting the patch mnually works!!
case 0x3: /* TTT TTTDD TTTDDD */ /* REVERT PATCH if (dp.dwCount > 4 && ((dp.dwFlags[3] & (DP_AM|DP_PM)) || (dp.dwFlags[4] & (DP_AM|DP_PM)) || (dp.dwFlags[5] & (DP_AM|DP_PM)))) hRet = DISP_E_TYPEMISMATCH; else */ if (dp.dwCount != 3 && dp.dwCount != 5 && dp.dwCount != 6) hRet = DISP_E_TYPEMISMATCH; st.wHour = dp.dwValues[0]; st.wMinute = dp.dwValues[1]; st.wSecond = dp.dwValues[2]; dwOffset = 3; dp.dwCount -= 3; break;
http://bugs.winehq.org/show_bug.cgi?id=22287
--- Comment #4 from Jaime jskartman@gmail.com 2010-04-06 07:11:18 --- ¿? i was editing and it goes submited 3 times while pressing tab ?
Anyway, manually reverting the patch on file /dlls/oleaut32/vartype.c works!
http://bugs.winehq.org/show_bug.cgi?id=22287
--- Comment #5 from André H. nerv@dawncrow.de 2010-04-07 12:48:00 --- Hi, thanks for spotting that. Tomorrow i should have time to look at it. On which locale do you run the app? do a: $ echo $LANG and print the output please
http://bugs.winehq.org/show_bug.cgi?id=22287
--- Comment #6 from Jaime jskartman@gmail.com 2010-04-08 01:50:17 --- My locale is spanish. es_ES.UTF-8
http://bugs.winehq.org/show_bug.cgi?id=22287
André H. nerv@dawncrow.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |ASSIGNED AssignedTo|wine-bugs@winehq.org |nerv@dawncrow.de Ever Confirmed|0 |1
--- Comment #7 from André H. nerv@dawncrow.de 2010-04-08 08:49:21 --- assigning to me
http://bugs.winehq.org/show_bug.cgi?id=22287
--- Comment #8 from André H. nerv@dawncrow.de 2010-04-08 09:36:29 --- Patch sent: http://source.winehq.org/patches/data/60278
Helping source for the future: ftp://ftp.software.ibm.com/software/globalization/locales/
http://bugs.winehq.org/show_bug.cgi?id=22287
André H. nerv@dawncrow.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution| |FIXED
--- Comment #9 from André H. nerv@dawncrow.de 2010-05-10 10:54:18 --- should be fixed by 7b66e1aeeb1cf871dc9a979675f69fcdb83a4a87
reopen otherwise
http://bugs.winehq.org/show_bug.cgi?id=22287
--- Comment #10 from Jaime jskartman@gmail.com 2010-05-10 12:07:46 --- Sorry, I can't test it. My locale is english now.
http://bugs.winehq.org/show_bug.cgi?id=22287
--- Comment #11 from André H. nerv@dawncrow.de 2010-05-10 12:43:04 --- Thats no problem, just run your app with:
cd path/to/your/app LANG=es_ES.UTF-8 wine yourapp.exe
http://bugs.winehq.org/show_bug.cgi?id=22287
--- Comment #12 from Jaime jskartman@gmail.com 2010-05-10 13:22:55 --- (In reply to comment #11)
Thats no problem, just run your app with:
cd path/to/your/app LANG=es_ES.UTF-8 wine yourapp.exe
That is working, but i dont have spanish locales installed so I am not sure that the program is not fallback to english ¿?
http://bugs.winehq.org/show_bug.cgi?id=22287
--- Comment #13 from André H. nerv@dawncrow.de 2010-05-10 13:30:37 --- looks good if it works, as wine always comes with all locales. so this is truly fixed. Thank you!
http://bugs.winehq.org/show_bug.cgi?id=22287
--- Comment #14 from Dmitry Timoshkov dmitry@codeweavers.com 2010-05-10 22:22:13 --- (In reply to comment #13)
looks good if it works, as wine always comes with all locales.
Not really. 'LANG=xx_YY' statement has no effect in WIne if the underlying system doesn't support particular locale.
http://bugs.winehq.org/show_bug.cgi?id=22287
--- Comment #15 from Dmitry Timoshkov dmitry@codeweavers.com 2010-05-10 22:26:48 --- Does 'LANG=es_ES.UTF-8 wine notepad' show the Notepad menus in Spanish?
http://bugs.winehq.org/show_bug.cgi?id=22287
--- Comment #16 from Jaime jskartman@gmail.com 2010-05-11 02:33:53 --- No, :(
http://bugs.winehq.org/show_bug.cgi?id=22287
Dmitry Timoshkov dmitry@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|FIXED |
--- Comment #17 from Dmitry Timoshkov dmitry@codeweavers.com 2010-05-11 02:49:56 --- Then the bug is not fixed. Do you have other locales besides en_US installed? Or can you install the es_ES locale for testing?
http://bugs.winehq.org/show_bug.cgi?id=22287
--- Comment #18 from Jaime jskartman@gmail.com 2010-05-11 03:00:42 --- I installed es_ES locales my notepad is spanish now.
The bug is not fixed I can see it with current git.
http://bugs.winehq.org/show_bug.cgi?id=22287
--- Comment #19 from Jaime jskartman@gmail.com 2010-05-11 03:09:55 --- The application is 11Mb i can send it via email, or to a ftp account. Let my know.
http://bugs.winehq.org/show_bug.cgi?id=22287
Dmitry Timoshkov dmitry@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|REOPENED |NEW
--- Comment #20 from Dmitry Timoshkov dmitry@codeweavers.com 2010-05-11 05:25:34 --- Thanks for testing.
http://bugs.winehq.org/show_bug.cgi?id=22287
--- Comment #21 from André H. nerv@dawncrow.de 2010-05-11 11:01:57 --- can you please run your app with WINEDEBUG=variant and attach the output? btw: the "revert" in comment #1 doesnt revert my code, but older code.
http://bugs.winehq.org/show_bug.cgi?id=22287
--- Comment #22 from Jaime jskartman@gmail.com 2010-05-25 06:46:32 --- I was doing the WINEDEBUG=variant with wine 1.2rc1 and all is working fine. I don't see the error anymore. I run notepad and it is in spanish so I am running chessimo in spanish too.
So seems fixed!
http://bugs.winehq.org/show_bug.cgi?id=22287
André H. nerv@dawncrow.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED
--- Comment #23 from André H. nerv@dawncrow.de 2010-05-26 07:36:32 --- as i said, it is fixed (WINEDEBUG only can have influence with relay on bug behaviors, not with normal channels)
http://bugs.winehq.org/show_bug.cgi?id=22287
--- Comment #24 from Austin English austinenglish@gmail.com 2010-05-26 12:29:19 --- (In reply to comment #23)
as i said, it is fixed (WINEDEBUG only can have influence with relay on bug behaviors, not with normal channels)
All channels can change bug behaviors. +heap/+snoop especially. Others channels can change code behavior, for instance, activating code hidden behind TRACE_ON(channel) see http://source.winehq.org/git/wine.git/?a=search&h=HEAD&st=grep&s...
http://bugs.winehq.org/show_bug.cgi?id=22287
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #25 from Alexandre Julliard julliard@winehq.org 2010-05-28 12:51:29 --- Closing bugs fixed in 1.2-rc2.
https://bugs.winehq.org/show_bug.cgi?id=22287
André H. nerv@dawncrow.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |7b66e1aeeb1cf871dc9a979675f | |69fcdb83a4a87 Regression SHA1| |71f24431b987d70053063a9f206 | |913e7fff53edc