[cfe-commits] r135614 - in /cfe/trunk: include/clang/Basic/DiagnosticDriverKinds.td include/clang/Driver/ArgList.h include/clang/Driver/Compilation.h include/clang/Driver/Driver.h include/clang/Driver/Job.h lib/Driver/ArgList.cpp lib/Driver/Compi

Chad Rosier mcrosier at apple.com
Thu Jul 21 09:22:26 PDT 2011


On Jul 20, 2011, at 11:29 PM, Frits van Bommel wrote:

> On 20 July 2011 22:26, Chad Rosier <mcrosier at apple.com> wrote:
>> +  // Free redirections of stdout/stderr.
>> +  if (Redirects) {
>> +    delete Redirects[1];
>> +    delete Redirects[2];
>> +    delete Redirects;
>> +  }
> 
> It's been a while since I've used dynamically-allocated arrays in C++,
> but shouldn't that last 'delete' be a 'delete[]'?

Fixed.  Thanks for the catch.

 Chad



More information about the cfe-commits mailing list