commit 48ba9ff3c44f33c65c26ecf5df306f4a428d2c26
author Chris Lattner <sabre@nondot.org> Thu Sep 24 20:15:51 2009 +0000
committer Chris Lattner <sabre@nondot.org> Thu Sep 24 20:15:51 2009 +0000
tree 8eb6d7253743c2aa98f16ed1e1f3653725b5fe1a
parent 07de8d1acf10117cac2cc53f1cb0ae888be6a5bc

tidy up, fix a memory leak in Regex::isValid


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@82707 91177308-0d34-0410-b5e6-96231b3b80d8

include/llvm/Support/Regex.h

diff --git a/include/llvm/Support/Regex.h b/include/llvm/Support/Regex.h
index 31fd3cc..4c4229e 100644
--- a/include/llvm/Support/Regex.h
+++ b/include/llvm/Support/Regex.h
@@ -54,7 +54,6 @@
     /// Matches.
     /// For this feature to be enabled you must construct the regex using
     /// Regex("...", Regex::Sub) constructor.
-
     bool match(const StringRef &String, SmallVectorImpl<StringRef> *Matches=0);
   private:
     struct llvm_regex *preg;

lib/Support/Regex.cpp

diff --git a/lib/Support/Regex.cpp b/lib/Support/Regex.cpp
index f8b2446..9f5fbb0 100644
--- a/lib/Support/Regex.cpp
+++ b/lib/Support/Regex.cpp
@@ -10,15 +10,15 @@
 // This file implements a POSIX regular expression matcher.
 //
 //===----------------------------------------------------------------------===//
+
 #include "llvm/Support/Regex.h"
 #include "llvm/Support/ErrorHandling.h"
 #include "llvm/Support/raw_ostream.h"
 #include "regex_impl.h"
 #include <string>
-
 using namespace llvm;
-Regex::Regex(const StringRef &regex, unsigned Flags)
-{
+
+Regex::Regex(const StringRef &regex, unsigned Flags) {
   unsigned flags = 0;
   preg = new struct llvm_regex;
   preg->re_endp = regex.end();
@@ -35,26 +35,23 @@
   error = llvm_regcomp(preg, regex.data(), flags|REG_EXTENDED|REG_PEND);
 }
 
-bool Regex::isValid(std::string &Error)
-{
+bool Regex::isValid(std::string &Error) {
   if (!error)
     return true;
 
   size_t len = llvm_regerror(error, preg, NULL, 0);
-  char *errbuff = new char[len];
-  llvm_regerror(error, preg, errbuff, len);
-  Error.assign(errbuff);
+  
+  Error.resize(len);
+  llvm_regerror(error, preg, &Error[0], len);
   return false;
 }
 
-Regex::~Regex()
-{
+Regex::~Regex() {
   llvm_regfree(preg);
   delete preg;
 }
 
-bool Regex::match(const StringRef &String, SmallVectorImpl<StringRef> *Matches)
-{
+bool Regex::match(const StringRef &String, SmallVectorImpl<StringRef> *Matches){
   unsigned nmatch = Matches ? preg->re_nsub+1 : 0;
 
   if (Matches) {
@@ -81,7 +78,7 @@
   // There was a match.
 
   if (Matches) { // match position requested
-    for (unsigned i=0;i<nmatch; i++) {
+    for (unsigned i = 0; i != nmatch; ++i) {
       if (pm[i].rm_so == -1) {
         // this group didn't match
         Matches->push_back(StringRef());