This PR renames srtp_octet_string_is_eq() to srtp_octet_string_equal() and switches the logic of the return value. That function returned false if equal, which was opposite of what the name suggests.
However, every test is actually looking for inequality. As such, perhaps a better name might be srtp_octet_string_unequal() or something? I don't care what we name this, but I'd prefer for the name reflect what it's doing.
This PR renames
srtp_octet_string_is_eq()
tosrtp_octet_string_equal()
and switches the logic of the return value. That function returnedfalse
if equal, which was opposite of what the name suggests.However, every test is actually looking for inequality. As such, perhaps a better name might be
srtp_octet_string_unequal()
or something? I don't care what we name this, but I'd prefer for the name reflect what it's doing.