Index: sources/canbus/MessageBuilder.cpp =================================================================== diff -u -r12e7b6dda53cf8db7707c7fa55dcf6137e7d6997 -r8d8dcd5a30aa0b465895788563b39a591b76d34b --- sources/canbus/MessageBuilder.cpp (.../MessageBuilder.cpp) (revision 12e7b6dda53cf8db7707c7fa55dcf6137e7d6997) +++ sources/canbus/MessageBuilder.cpp (.../MessageBuilder.cpp) (revision 8d8dcd5a30aa0b465895788563b39a591b76d34b) @@ -208,8 +208,6 @@ vActual = vData.back(); vExpected = calcCRC(vData.mid(0, len - 1)); bool ok = vExpected == vActual; - // it's very good but I'm not sure if it's correct. - //bool ok = calcCRC(vData) == 0; return ok; #else return true; Index: sources/canbus/MessageGlobals.h =================================================================== diff -u -r37e62f8f5c0345cd65b5e4161f07e86b442c22ac -r8d8dcd5a30aa0b465895788563b39a591b76d34b --- sources/canbus/MessageGlobals.h (.../MessageGlobals.h) (revision 37e62f8f5c0345cd65b5e4161f07e86b442c22ac) +++ sources/canbus/MessageGlobals.h (.../MessageGlobals.h) (revision 8d8dcd5a30aa0b465895788563b39a591b76d34b) @@ -139,9 +139,9 @@ eLenActionId = 2, ///< The length of MessageID bytes eLenLength = 1, ///< The length of data length value byte at the beginning of each header frame after MessageID - eLenCRCDigits = 2, - eLenChannelDigits = 3, - eLenMessageIDDigits = 4, + eLenCRCDigits = 2, ///< The length of CRC value byte in each Denali message + eLenChannelDigits = 3, ///< The length of channel value byte in each Denali message + eLenMessageIDDigits = 4, ///< The length of message id value byte in each Denali message };