You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I spoke to the team responsible for this and they say that this code is legit; apparently it’s more optimal to use this rather than the logical AND. It has the same effect but it’s not really intuitive. Is it causing a compiler warning? If so it should be fixed. At the very least, it should be commented to indicate that it is intended behavior. I'll update this post when the fix has been checked in
Source code is
if (IsOutOfBandMessage(message) & !IsValidOutOfBandMessage(message))
Maybe better code
if (IsOutOfBandMessage(message) && !IsValidOutOfBandMessage(message))
The text was updated successfully, but these errors were encountered: