Extending the testframe to cope with different windows versions

Paul Vriens paul.vriens.wine at gmail.com
Sat Nov 25 09:58:16 CST 2006


On Sat, 2006-11-25 at 07:33 -0800, James Hawkins wrote:
> The if 0 exclusion is enough because it shows that at one point
> advpack checked for the null parameter, but now it doesn't on some
> versions.  We check for the null param, so we're on the safe side of
> the fence.  Not many apps/installers depend on a certain function
> crashing with a null param, though there are one or two we've seen.
> advpack is almost completely undocumented and is only used by the IE
> and mdac installers (basically), so this issue is inconsequential.
> Also, Alexandre has specifically said that he doesn't want the tests
> filled with version checks.
> 
The one mentioned is maybe the wrong example as it's quite clear. The
point however is that this test will never be run anymore as long as
nobody cares.

Cheers,

Paul.




More information about the wine-devel mailing list