Fix some comments

Author: peternewman

common/rdm/ResponderTagSetTest.cpp

@@ -65,9 +65,12 @@ void TagSetTest::testTagSet() {
   OLA_ASSERT_EQ(string("foo"), set1.ToString());
   OLA_ASSERT_TRUE(set1.Contains(tag));
   OLA_ASSERT_FALSE(set1.Contains(tag2));
+  // Add the same tag again (shouldn't change anything)
   set1.AddTag(tag);
-  OLA_ASSERT_TRUE(set1.Contains(tag));
   OLA_ASSERT_EQ(1u, set1.Size());
+  OLA_ASSERT_EQ(string("foo"), set1.ToString());
+  OLA_ASSERT_TRUE(set1.Contains(tag));
+  OLA_ASSERT_FALSE(set1.Contains(tag2));
 
   // now test the packing
   unsigned int buffer_size2 = 32 + 1;

include/ola/rdm/ResponderTagSet.h

@@ -164,8 +164,9 @@ class TagSet {
       std::ostringstream str;
       std::set<std::string>::const_iterator iter;
       for (iter = m_tags.begin(); iter != m_tags.end(); ++iter) {
-        if (iter != m_tags.begin())
+        if (iter != m_tags.begin()) {
           str << ",";
+        }
         str << *iter;
       }
       return str.str();

include/ola/rdm/UIDSet.h

@@ -206,8 +206,9 @@ class UIDSet {
       std::ostringstream str;
       std::set<UID>::const_iterator iter;
       for (iter = m_uids.begin(); iter != m_uids.end(); ++iter) {
-        if (iter != m_uids.begin())
+        if (iter != m_uids.begin()) {
           str << ",";
+        }
         str << *iter;
       }
       return str.str();