The messageID for CAN constructor needs to be unsigned to avoid unnecessary implicit casting

pull/7848/head
Aashish chaddha 2018-08-21 12:32:22 -05:00
parent b53a9ea4c0
commit c07e29bd2a
1 changed files with 2 additions and 2 deletions

View File

@ -55,7 +55,7 @@ public:
* @param _type Type of Data: Use enum CANType for valid parameter values * @param _type Type of Data: Use enum CANType for valid parameter values
* @param _format Data Format: Use enum CANFormat for valid parameter values * @param _format Data Format: Use enum CANFormat for valid parameter values
*/ */
CANMessage(int _id, const char *_data, char _len = 8, CANType _type = CANData, CANFormat _format = CANStandard) CANMessage(unsigned _id, const char *_data, char _len = 8, CANType _type = CANData, CANFormat _format = CANStandard)
{ {
len = _len & 0xF; len = _len & 0xF;
type = _type; type = _type;
@ -69,7 +69,7 @@ public:
* @param _id Message ID * @param _id Message ID
* @param _format Data Format: Use enum CANType for valid parameter values * @param _format Data Format: Use enum CANType for valid parameter values
*/ */
CANMessage(int _id, CANFormat _format = CANStandard) CANMessage(unsigned _id, CANFormat _format = CANStandard)
{ {
len = 0; len = 0;
type = CANRemote; type = CANRemote;