[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