Github user astitcher commented on the issue:
https://github.com/apache/qpid-proton/pull/160
I definitely like making option<T> a class in common rather than
duplicating it.
I think that this is the correct way to expose options in the API, but
perhaps we could make the API a bit more easy to use. Perhaps we could have a
C++11 only explicit bool conversion as well as is_set(). However we would need
to experiment a bit as some of the options themselves are also bools so this
might be confusing!
Another thing we should consider is throwing an exception if get() is
called when the option is not set.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
