r210075 - Don't assume an implicit IntrusiveRefCntPtr -> bool operator.
Rafael EspĂndola
rafael.espindola at gmail.com
Tue Jun 3 06:38:45 PDT 2014
>> - bool hasInvocation() const { return Invocation != nullptr; }
>> + bool hasInvocation() const { return (bool)Invocation; }
>
> Perhaps IntrusiveRefCntPtr could ues an operator==(nullptr_t) though?
Sure, r210100 and r210101.
> (& can we move from IntrusiveRefCntPtr to std::shared_ptr with
> std::enable_shared_from_this where necessary?)
Maybe. As a current side project I am trying to change llvm to use
std::errror_code, so I will not be looking at IntrusiveRefCntPtr right
now, but if you have the time it would be nice to try.
Cheers,
Rafael
More information about the cfe-commits
mailing list