Small binary native spec nit.

[frederik...@gmail.com]

Hi,

I believe http://www.blinkprotocol.org/s/BlinkNativeSpec-beta4.pdf contains a minor error  in section "3.14 Dynamic Group", the binary given as example ends with:

 03 00 00 00 // Rad: 2

I believe the comment should read "// Rad: 3" instead.

Thanks,

Frederik

[David Rosenborg]

Good catch, it should be 3 indeed. A fix will be included when we release a new version of the spec next time.

Thanks

/David

--
You received this message because you are subscribed to the Google Groups "The Blink Protocol" group.
To unsubscribe from this group and stop receiving emails from it, send an email to the-blink-proto...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/the-blink-protocol/f779651e-c802-4bf0-ab7d-05ebef0442c6%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

[Frederik Deweerdt]

Also in the same example:

 13 00 00 00 // Shapes [1]: offset: 19 (3)

I believe should read

 18 00 00 00 // Shapes [1]: offset: 24 (3)

Thanks,

Frederik

[David Rosenborg]

It’s even

1c 00 00 00 // Shapes [1]: offset: 28 (3)

since the size of the offset field itself is included in the offset.

Thanks

/David

To view this discussion on the web visit https://groups.google.com/d/msgid/the-blink-protocol/CAPYV75NcBriTxXULoAz9i8cTaYEVjD7bT67-b6LsMxc9gYMb5g%40mail.gmail.com.