[PATCH] [libcxx] Rework sized delete.
Marshall Clow
mclow.lists at gmail.com
Mon May 18 13:07:29 PDT 2015
I think that the language specific files could be named `sized_delete_array11` and `sized_delete_array14` instead of using the `++` in the file names.
Other than that (and the inline nit), these look good to me.
================
Comment at: test/std/language.support/support.dynamic/new.delete/new.delete.array/sized_delete_array_calls_unsized_delete_array.pass.cpp:54
@@ +53,3 @@
+ delete [] x;
+ assert(delete_called == 1);
+ assert(!delete_nothrow_called);
----------------
Humor my OCD, please.
Make these match all the others:
`1 == delete_called` and `0 == delete_nothrow_called`
http://reviews.llvm.org/D9831
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the cfe-commits
mailing list