Re: [boost] [PATCH Boost.smart_ptr] Add intrusive_ptr<>::release()

On Tue, Dec 3, 2013 at 3:45 AM, Avi Kivity wrote:

On Mon, Dec 2, 2013 at 8:45 PM, Peter Dimov wrote:

...

Andrey Semashev wrote:

...

The reset/release pattern is quite established in STL and Boost and I'd prefer not to disturb it. This will only add confusion.

I agree with Lars and Jonathan, sorry Andrey. Naming the function "release" will add confusion even though you might argue that it will improve consistency.

I'll rename it to detach() if that's the consensus.

Concern over misuse was a key reason past release() proposals didn't fly(). Changing the name would help, but an explicit caution about misuse in the docs might also help. Thanks for including doc and test patches. That always makes a good impression:-) --Beman