Adding size getter for message.

This wasn't done before and was an oversight. This fixes #19 issue.
I am not sure if uint8_t is the right size but if it is greater rollover happens and no detection method right now

Author: LinuxDevon

include/libdbc/message.hpp

@@ -32,9 +32,8 @@ struct Message {
 	void appendSignal(const Signal& signal);
 	const std::vector<Signal> getSignals() const;
 	uint32_t id() const;
-	const std::string& name() const {
-		return m_name;
-	}
+	uint8_t size() const;
+	const std::string& name() const;
 	void addValueDescription(const std::string& signal_name, const std::vector<Signal::SignalValueDescriptions>&);
 
 	virtual bool operator==(const Message& rhs) const;

src/message.cpp

@@ -85,6 +85,14 @@ uint32_t Message::id() const {
 	return m_id;
 }
 
+uint8_t Message::size() const {
+	return m_size;
+}
+
+const std::string& Message::name() const {
+	return m_name;
+}
+
 void Message::addValueDescription(const std::string& signal_name, const std::vector<Signal::SignalValueDescriptions>& vd) {
 	for (auto& s : m_signals) {
 		if (s.name.compare(signal_name) == 0) {

test/test_dbc.cpp

@@ -73,6 +73,7 @@ TEST_CASE("Testing  big endian, little endian") {
 
 	REQUIRE(parser.get_messages().size() == 1);
 	REQUIRE(parser.get_messages().at(0).name() == "MSG1");
+	REQUIRE(parser.get_messages().at(0).size() == 8);
 	REQUIRE(parser.get_messages().at(0).getSignals().size() == 2);
 	{
 		const auto signal = parser.get_messages().at(0).getSignals().at(0);
@@ -85,7 +86,7 @@ TEST_CASE("Testing  big endian, little endian") {
 }
 
 TEST_CASE("Testing negative values") {
-	std::string dbc_contents = PRIMITIVE_DBC + R"(BO_ 234 MSG1: 8 Vector__XXX
+	std::string dbc_contents = PRIMITIVE_DBC + R"(BO_ 234 MSG1: 58 Vector__XXX
  SG_ Sig1 : 55|16@0- (0.1,0) [-3276.8|-3276.7] "C" Vector__XXX
  SG_ Sig2 : 39|16@0- (0.1,0) [-3276.8|-3276.7] "C" Vector__XXX
  SG_ Sig3 : 23|16@0- (10,0) [-3276.8|-3276.7] "C" Vector__XXX
@@ -97,6 +98,7 @@ TEST_CASE("Testing negative values") {
 
 	REQUIRE(parser.get_messages().size() == 1);
 	REQUIRE(parser.get_messages().at(0).name() == "MSG1");
+	REQUIRE(parser.get_messages().at(0).size() == 58);
 	REQUIRE(parser.get_messages().at(0).getSignals().size() == 4);
 
 	SECTION("Evaluating first message") {
@@ -130,7 +132,7 @@ TEST_CASE("Testing negative values") {
 }
 
 TEST_CASE("Special characters in unit") {
-	std::string dbc_contents = PRIMITIVE_DBC + R"(BO_ 234 MSG1: 8 Vector__XXX
+	std::string dbc_contents = PRIMITIVE_DBC + R"(BO_ 234 MSG1: 255 Vector__XXX
  SG_ Speed : 0|8@1+ (1,0) [0|204] "Km/h"  DEVICE1,DEVICE2,DEVICE3)";
 	const auto filename = create_temporary_dbc_with(dbc_contents.c_str());
 
@@ -139,6 +141,7 @@ TEST_CASE("Special characters in unit") {
 
 	REQUIRE(parser.get_messages().size() == 1);
 	REQUIRE(parser.get_messages().at(0).name() == "MSG1");
+	REQUIRE(parser.get_messages().at(0).size() == 255);
 	REQUIRE(parser.get_messages().at(0).getSignals().size() == 1);
 	SECTION("Checking that signal with special characters as unit is parsed correctly") {
 		const auto signal = parser.get_messages().at(0).getSignals().at(0);