properties() method of ObjectHandler::Object class.

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

properties() method of ObjectHandler::Object class.

Slava Mazur
Greetings,

I noticed that method properties() in ObjectHandler::Object class
declared as non-const:
const boost::shared_ptr<ValueObject>& properties();

Is it intended? If yes, could someone tell me the reason?
Wouldn't it better to have the following?

const boost::shared_ptr<ValueObject>& properties() const;

Thanks,

Slava


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
QuantLib-dev mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/quantlib-dev
Reply | Threaded
Open this post in threaded view
|

Re: properties() method of ObjectHandler::Object class.

Eric Ehlers-2
Hi Slava,

On Wed, March 19, 2008 6:25 pm, Slava Mazur wrote:

> Greetings,
>
> I noticed that method properties() in ObjectHandler::Object class
> declared as non-const:
> const boost::shared_ptr<ValueObject>& properties();
>
> Is it intended? If yes, could someone tell me the reason?
> Wouldn't it better to have the following?
>
> const boost::shared_ptr<ValueObject>& properties() const;

You're right, that function ought to be const, and making it so seems not
to break anything.  I made the change in subversion for inclusion in the
next release.  Thank you.

Regards,
Eric



-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
QuantLib-dev mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/quantlib-dev