3 ms·
Wait.. are you saying that the conditional was actually: if (isSignatureValid == isSignatureFromGuardian) As variable names, “validSignature” and “signatu
by mewse 5y ago
Wait.. are you saying that the conditional was actually:
if (isSignatureValid == isSignatureFromGuardian)
As variable names, “validSignature” and “signatureFromGuardian” sound like they’re storing the literal signature values, but from your description these are actually holding boolean results _describing_ the signatures, and not the signatures themselves?
Geez, if you name variables that it’d be amazingly easy to make a mistakes like what you describe, and then just read straight over them when you’re reviewing the code.