shell32/tests: add tests for the parser of SHELLEXECUTEINFO.lpFile [try 4]
Paul Vriens
paul.vriens.wine at gmail.com
Mon Mar 8 10:49:47 CST 2010
On 03/08/2010 05:40 PM, Ilya Basin wrote:
>>> JL> Also, those TEST_LPFILE_PARSED_OK macros are ugly, and hard to fix if
>>> JL> one place has a certain behavior, while another has different
>>> JL> behavior. Just use ok directly wherever you're using the macro.
>>> Not convinced. What you say is good for reading, but not editing.
>>> If someone wants to change behavior , the other person
>>> defined in a macro, but only in one place, he can jast as simple
>>> copy-paste the definition and edit it.
>>> In any case, it's a free country (c).
>
> PV> I must agree with Juan here.
>
> PV> Apart from that, the macro TEST_LPFILE_PARSED_ok_condition is strange in
> PV> itself as it checks for 'expected' and this is something you always set
> PV> yourself.
>
> PV> I agree with you that it's a free country and stuff like this will
> PV> always be up for debate but in the end it's AJ that commits the patches
> PV> and he has expressed several times not being to fond about too much macros.
>
> If it's well known, you had to tell me about macros at the beginning.
This is not a general rule, just something I noticed lately.
> Sent try 6.
Thanks for your dedication.
--
Cheers,
Paul.
More information about the wine-devel
mailing list