"new"->"NEW" in comments, easier to find remaining naked operator new

src/xml/XMLTagHandler.cpp

@@ -214,7 +214,7 @@ bool XMLTagHandler::ReadXMLTag(const char *tag, const char **attrs)
    }
 
 // JKC: Previously the next line was:
-// const char **out_attrs = new char (const char *)[tmp_attrs.GetCount()+1];
+// const char **out_attrs = NEW char (const char *)[tmp_attrs.GetCount()+1];
 // however MSVC doesn't like the constness in this position, so this is now
 // added by a cast after creating the array of pointers-to-non-const chars.
    const wxChar **out_attrs = (const wxChar**)new wxChar *[tmp_attrs.GetCount()+1];

src/xml/XMLTagHandler.h

@@ -91,7 +91,7 @@ class AUDACITY_DLL_API XMLTagHandler {
    virtual void HandleXMLContent(const wxString & WXUNUSED(content)) {}
 
    // If the XML document has children of your tag, this method
-   // should be called.  Typically you should construct a new
+   // should be called.  Typically you should construct a NEW
    // object for the child, insert it into your own local data
    // structures, and then return it.  If you do not wish to
    // handle this child, return NULL and it will be ignored.