02-21-2020, 08:43 PM
Oh wait I think I was misunderstanding and thinking that returning 0 indicated success. But now I'm thinking maybe returning 1 is success and 0 means failure. So in that case that check you highlighted is just so early in the function because if those bytes are not set correctly at that point, there's no point going any further and checking the checksum etc. Is that about right?