From c07e29bd2aae8542c8c21ed1f5f92756e4d738c2 Mon Sep 17 00:00:00 2001 From: Aashish chaddha Date: Tue, 21 Aug 2018 12:32:22 -0500 Subject: [PATCH] The messageID for CAN constructor needs to be unsigned to avoid unnecessary implicit casting --- drivers/CAN.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/CAN.h b/drivers/CAN.h index 5f506f68ad..c1aae5c771 100644 --- a/drivers/CAN.h +++ b/drivers/CAN.h @@ -55,7 +55,7 @@ public: * @param _type Type of Data: Use enum CANType 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; type = _type; @@ -69,7 +69,7 @@ public: * @param _id Message ID * @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; type = CANRemote;