Why would you add all your findings and "test cases" into the Wine's test suite? Until then you should stop flaming list. With all the energy you spent writing these e-mails BACKWARDS as I might add.
And you outright IGNORED my suggestions on the IRC chat room.
So until you show us some code that verifies what you are talking about, please stay away from the e-mail clinet.
Vitaliy.
Monday, June 5, 2006, 10:05:39 AM, EA Durbin wrote:
It doesn't use %d currently. It uses %i. And its not to print. It uses this value in the SQL statement. The LastSequence value of the Media table is NEVER negative as i pointed out in the MSDN link. It must always be zero or larger and this is handled by passing %u.
The SELECT * FROM `Media` WHERE `LastSequence` >= %i, should be The SELECT * FROM `Media` WHERE `LastSequence` >= %u. In ready_media_for_files.
From: Mike McCormack mike@codeweavers.com To: EA Durbin ead1234@hotmail.com CC: dank@kegel.com, wine-devel@winehq.org, hans@it.vu.nl Subject: Re: msi: Fix some copy/paste bugs in the implementation of condition operators. Date: Tue, 06 Jun 2006 00:38:16 +0900
EA Durbin wrote:
I did in fact write a test, which does prove this. Hans has a copy of it, and i think i sent it to wine-devel too.
I haven't seen any patch for the Wine regression test suite as yet, and that is what I have been asking you to write.
As for files.c, the query in ready_media_for_file() should be passing an unsinged integer as LastSequence must be greater than zero.
You patch is wrong because the value you are printing is an INT, so using %d is correct, and using %u will print the wrong thing when the INT is less than zero.
Mike