@elasota : Couple of nitpicks, could you edit the commit message to remove the trailing fullstop and add the reason for the issue i.e. Window build fails due to this.

Beyond that, I was under the impression that comparing bool to int, while not good style is valid C/C++ syntax.

Thanks!

Beyond that, I was under the impression that comparing bool to int, while not good style is valid C/C++ syntax.

It is valid but it was promoted to an error because it is usually a mistake. Comparing a bool to an int does not coerce the int to bool by comparing it to zero, it coerces the bool to the scalar type and compares it as an integer, meaning it will only compare true if the int value is equal to the integer value of "true" (which is usually 1, but is actually implementation-defined, which is one more reason to not do this).

Thanks for the fix! Among all the commits from upstream I didn't spot this misleading constuct :)