Re: [PATCH]: tentative fix for libsvn_ra_dav request_auth-bug (issue 1005)

cmpilato@collab.net writes:

> I *refuse* to apply this patch...because ***I just wrote almost the
> exact same code myself!***
>
> :-)
>
> I mean, the only difference I can see is that you use:
>
> if (foo == NULL)
>
> and I use:
>
> if (! foo)
>
> Wow.

Hehe. Well, yeah, it was mostly a no-brainer. Not too many ways to
do it really.

The reason I opted for (foo == NULL) over (! foo) was that NULL in
this case is a kind of "magic value", so it made sense to compare
against it explicitly to better show what was going on. It doesn't
matter much either way.

  // Marcus

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Nov 22 21:37:45 2002