Merge pull request #355 from pklampros/keycolumn_name

Make sure the key column actually gets a name
SpaceGroupUCL · Jul 9, 2020 · 776b128 · 776b128
2 parents 6d31883 + 7118ad2
commit 776b128
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/salalib/attributetable.h b/salalib/attributetable.h
@@ -26,6 +26,12 @@
 #include <salalib/displayparams.h>
 #include <salalib/mgraph_consts.h>
 
+///
+/// Namespace to hold known attributes
+///
+namespace AttributeName {
+    const char * const REF = "Ref";
+}
 
 ///
 /// Interface to an attribute row
@@ -160,10 +166,7 @@ class AttributeColumnImpl: public AttributeColumn, AttributeColumnStats
 class KeyColumn : public AttributeColumnImpl
 {
 public:
-    KeyColumn() : AttributeColumnImpl(), m_name("Ref")
-    {}
-private:
-    std::string m_name;
+    KeyColumn() : AttributeColumnImpl() { setName(AttributeName::REF); }
 };