Confusing // Now, fill buf[1 .. 63] with whatever data you want. in documentation

[redfast00]

In the README, under encoding tiny frames there is a comment // Now, fill buf[1 .. 63] with whatever data you want.

When I first read this, it wasn't entirely clear what was meant by this; by reading the code, I understand that this 1 .. 63 is inclusive of the beginning, but not of the end (so you can't fill in buf[63]). Maybe this could be made clearer:

// Now, fill in buf[1] up to and including buf[62] (so 64 - 2 = 62 bytes of payload data)

[charlesnicholson]

Ah, great catch, thanks! Your assessment is correct; it's an off-by-one bug in the docs and not anything deeper in the library :). Sorry about the confusion! I'll fix it asap, or if you want to open a PR against it I'll happily take it. (I think you can do that entirely from the UI for readme changes)

[redfast00]

I'lll open a PR :)

[charlesnicholson]

Add your (user)name to the contributors list too if you like!

[redfast00]

Thanks :) By the way, I like what you did with the new naming for tinyframe, it really makes it clear what it's intended for.

[charlesnicholson]

Thanks, it was in retrospect just truly awful naming that confused everyone. Sigh.