================ @@ -412,7 +412,7 @@ class DerefClass{ void testDoubleDeleteClassInstance() { DerefClass *foo = new DerefClass(); delete foo; - delete foo; // newdelete-warning {{Attempt to delete released memory}} + delete foo; // newdelete-warning {{Attempt to free released memory}} ---------------- NagyDonat wrote:
I see your point and agree that _in that particular case_ the spelling with `free` is slightly worse than the old message that said `delete`, but I think that overall the _consistently_ slightly worse message is still better than the inconsistent situation where the checker could emit different messages in almost-identical situations. I'll try to ensure that in the foreseeable future _somebody_ (e.g. an intern in our team) systemically updates the messages to use properly generic vocabulary and e.g. say "release" instead of "free". https://github.com/llvm/llvm-project/pull/147542 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits