[PATCH 5/6] urlmon: Added user agent tests.

Paul Vriens paul.vriens.wine at gmail.com
Wed Jul 1 00:50:49 CDT 2009


Jacek Caban wrote:
> ---
>  dlls/urlmon/tests/misc.c |   84 
> ++++++++++++++++++++++++++++++++++++++++++++-
>  1 files changed, 82 insertions(+), 2 deletions(-)
> 
> 
> 
> ------------------------------------------------------------------------
> 
> 
Hi Jacek,

The addition of these new tests led to a Coverity warning (#947) as your 
using str2 in your tests. str2 is checked for NULL earlier on but only 
skips a few tests now if it's indeed NULL.

The question is, should we just add a return after the skip() or get rid 
of the whole checking for NULL.

(The str2 checking was not added by you btw).

-- 
Cheers,

Paul.



More information about the wine-devel mailing list