const get methods in IAttributes

3D Engine c++

Brought to you by: cutealien, nimphoenix2

#311 const get methods in IAttributes

Milestone: SVN trunk development

Status: closed-accepted

Owner: Michael Zeilfelder

Labels: None

Priority: 5

Updated: 2016-03-27

Created: 2016-03-25

Creator: Erik Schultheis

Private: No

the getAttributeAs... functions in IAttributes are not const, except curiously in the case for int. simply adding const to all method declarations seems to work, i.e. the implementations are not doing any non-const accesses and as such it seems reasonable to declare these functions const.

1 Attachments

attributes_const.patch

Discussion

Michael Zeilfelder - 2016-03-27

status: open --> closed-accepted

assigned_to: Michael Zeilfelder
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Michael Zeilfelder - 2016-03-27

Yeah, I also see no reason for them not to be const. Thanks for the patch, it's been applied in svn trunk r5293

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

const get methods in IAttributes

3D Engine c++

Group

Searches

Help

#311 const get methods in IAttributes

Discussion