This will indeed hide the warning, but it's not changing the semantics meaning that the if will always trigger nd the branch will always be taken.

If that is what we want, then we should remove the if. If it is not what we want, then we should change it. Making the code confusing enough that the compiler is just as confused as the programmer is not really the fix I was after.