Re: wininet: InternetGetConnectedStateExW must return TRUE for success (try 2)

[Henri Verbeet]

On 29 January 2014 23:01, Bruno Jesus <00cpxxx(a)gmail.com> wrote:
> Citavi relies on == TRUE comparison so return TRUE instead of != FALSE

I think that in general this kind of change benefits from a comment
indicating that the extra compare is on purpose, mostly to avoid
someone from removing it again in the future.

wininet: InternetGetConnectedStateExW must return TRUE for success (try 2)

[Bruno Jesus]

On Thu, Jan 30, 2014 at 8:22 AM, Henri Verbeet <hverbeet(a)gmail.com> wrote:
> On 29 January 2014 23:01, Bruno Jesus <00cpxxx(a)gmail.com> wrote:
> > Citavi relies on == TRUE comparison so return TRUE instead of != FALSE
>
> I think that in general this kind of change benefits from a comment
> indicating that the extra compare is on purpose, mostly to avoid
> someone from removing it again in the future.

OK, I'll send an updated version later.

Thanks,
Bruno