mysmartgrid / hexabus

The HexaBus Project: An IPv6-based home automation bus. We develop both hard- and software for the future of home automation.
http://hexabus.net
49 stars 15 forks source link

HXB_DTYPE_66BYTES #228

Closed morriswinkler closed 10 years ago

morriswinkler commented 10 years ago

Cheers mate,

i just figured in shared/hexabus_definitions.h

#define HXB_66BYTES_PACKET_MAX_BUFFER_LENGTH 65

so the type name and the value doesn't match, after i asked on the #hexabus channel if this is intended ` sad that this is a typo, and that 66BYTES needs to be renamed 65BYTES since that payload is used from the statemachine upload, which is 65 Bytes.

So that means HXB_DTYPE_66BYTES needs also to be renamed to HXB_DTYPE_65BYtes.

While i am ad it, why is this called MAX_BUFFER_LENGTH when a 66Byte payload needs always to be 65 bytes long and can't be 64 bytes in length.

I know that are just internals but that is confusing, since MAX_BUFFER_LENGTH indicates that that buffer can be smaller. Same for 16Bytes and String.

morriswinkler commented 10 years ago

nice :)