Menu

#43 Field<string> value accessor should return reference

v0.3.18
accepted
nobody
5
2017-11-09
2014-12-23
Anonymous
No

Line 128 of Field.hpp is

std::string value() const { return _value; }

The line should be changed to return a reference?

std::string& value() const { return _value; }

Discussion

  • Anonymous

    Anonymous - 2014-12-23

    Sorry, just realized the change should be to

    const std::string& value() const { return _value; }

    (forgot the const)

     
  • Gulliver

    Gulliver - 2014-12-24
    • status: open --> accepted
     
  • Gulliver

    Gulliver - 2017-11-09
    • Group: v0.3.17 --> v0.3.18
     

Anonymous
Anonymous

Add attachments
Cancel





Auth0 Logo