From f1ef53bea0447aba1d5bdc5c69c1b7ec9e8dae4a Mon Sep 17 00:00:00 2001 From: Paul Beckingham Date: Sat, 4 Apr 2015 10:58:55 -0400 Subject: [PATCH] Column: Return const refs rather than string copies --- src/columns/Column.h | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/columns/Column.h b/src/columns/Column.h index 3a6662d9e..e59547c3d 100644 --- a/src/columns/Column.h +++ b/src/columns/Column.h @@ -46,10 +46,10 @@ public: bool operator== (const Column&) const; // TODO Is this necessary? virtual ~Column (); - std::string name () const { return _name; } - std::string style () const { return _style; } - std::string label () const { return _label; } - std::string type () const { return _type; } + const std::string& name () const { return _name; } + const std::string& style () const { return _style; } + const std::string& label () const { return _label; } + const std::string& type () const { return _type; } bool modifiable () const { return _modifiable; } bool is_uda () const { return _uda; } std::vector styles () const { return _styles; }