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

[Jacek Caban]

Hi Paul,

Paul Vriens wrote:
> 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.

Checking memory allocation in tests doesn't make sense IMO. I think 
existing checking for NULL should be removed.


Thanks,
    Jacek