[PATCH] Deprecate and remove OwningPtr

Chandler Carruth chandlerc at google.com
Thu Jun 19 00:03:05 PDT 2014


On Thu, Jun 19, 2014 at 7:53 AM, Craig Topper <craig.topper at gmail.com>
wrote:

> On Wed, Jun 18, 2014 at 11:33 PM, Aaron Ballman <aaron at aaronballman.com>
> wrote:
>
>> On Thu, Jun 19, 2014 at 1:30 AM, Alp Toker <alp at nuanti.com> wrote:
>> > The attached patch slaps some deprecation attributes on the OwningPtr
>> > classes and removes associated unit tests.
>> >
>> > I'll follow up and remove the OwningPtr.h header completely a couple of
>> days
>> > after to provide a last-chance grace period.
>>
>> "A couple of days" is not exactly a grace period, especially given how
>> many folks are at the standards meetings currently. Was this grace
>> period discussed at some point?
>>
>
> I believe most in tree uses of OwningPtr were removed in March or April.
> The last in tree usage was removed by me on May 21. Hopefully by now anyone
> even sort of watching the list would know that OwningPtr's days were
> numbered.
>

I think we've given plenty of grace period. Just remove it, no need to jump
through deprecation hoops.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140619/84c78c0b/attachment.html>


More information about the llvm-commits mailing list