On Fri, May 15, 2015 at 5:31 AM, Ismail Pazarbasi
<[email protected]> wrote:
> Reduced test case:
>   struct C { };
>
>   struct A {
>     C *p;
>     A() : p{nullptr} { }
>     ~A() { delete p; }
>   };
>
> This is an oversight. I will fix it by replacing:
>   return dyn_cast<const CXXNewExpr>(E);
> with:
>   return dyn_cast_or_null<const CXXNewExpr>(E);

Great. My reduce job ended up with something similar:

class Cord;
class A {
  A(const A &);
  Cord *saved_cord_{nullptr};
};
A::A(const A &) { delete saved_cord_; }

Are you going to add it to the testsuite together with your patch, or
do you want me to add it first?


Thanks.  Diego.
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to