[PATCH 1/2] quartz/tests: move shareable code to a separate file

James McKenzie jjmckenzie51 at earthlink.net
Mon Aug 23 20:45:06 CDT 2010


Anton Khirnov wrote:
> On Sat, Aug 21, 2010 at 07:50:50PM -0700, James McKenzie wrote:
>   
>> Looks like you did not test the build before running it against the
>> testbot (I've been guilty of this as well.)
>>
>>     
> actually i did test it and it worked for some reason, must be magic :)
> in any case, after rebasing it doesn't -- and the reason seems to be
> that the build system assumes that each .c file under tests/ actually
> contains a test. is there a way around that (besides the
> obvious-but-ugly empty test in the common file)?
>
>   

One hint is if you are introducing a new function that exists in MSDN, 
look for an existing test case.  If there is none, one will be required 
before Wine implementation code will be accepted.  If the test exists in 
a file, then the implementing code should go in the corresponding .c 
file in the base dll directory from what I've read here over the years.

If you are introducing a new function class, then it may be best to put 
the tests and code into new files.  Empty test case files are frowned 
upon unless there really is no way to successfully test what you are 
trying to implement, but that is extremely rare.

It is best to ask in the Wine Development list as to the most 
appropriate manner of implementation.  The folks in the list are much 
more knowledgeable about this, I'm just giving suggestions.

James McKenzie



More information about the wine-devel mailing list