REGRESSION (r49213): http/tests/plugins/local-geturl-from-remote.html is failing on Windows

[Adam Roben (:aroben)]

Created attachment 40788 [details]
Patch v1

r49213 contained a typo that is causing http/tests/plugins/local-geturl-from-remote.html to fail on Windows.

[Adam Roben (:aroben)]

Committed r49244: <http://trac.webkit.org/changeset/49244>

[Adam Barth]

Thanks for fixing this.  How did the old code even compile?

[Alexey Proskuryakov]

> How did the old code even compile?

It's operator comma, which is often used in for loops, but is universally available in fact:

for (i =0, j = 0; i != 1; i++, j++) {...}

The value of such an expression is the value of the last sub-expression. Unless I'm hallucinating, the ChangeLog comment here got it wrong (the condition was always false, not always true).

[Adam Barth]

Wow, that's ridiculous.  Thanks for explaining.

[Adam Roben (:aroben)]

(In reply to comment #3)
> Unless
> I'm hallucinating, the ChangeLog comment here got it wrong (the condition was
> always false, not always true).

The ChangeLog was wrong.