用内存泄漏工具检查到这两个地方有内存泄漏

[jimmy54]

recognizer.cc 25行

static void load_patterns(RecognizerPatterns* patterns, an<ConfigMap> map) {
  if (!patterns || !map)
    return;
  for (auto it = map->begin(); it != map->end(); ++it) {
    auto value = As<ConfigValue>(it->second);
    if (!value)
      continue;
    (*patterns)[it->first] = value->str();  //这里有内存泄漏
  }
}

void RecognizerPatterns::LoadConfig(Config* config) {
  an<ConfigMap> pattern_map;
  string preset;
  if (config->GetString("recognizer/import_preset", &preset)) {
    the<Config> preset_config(
        Config::Require("config")->Create(preset));
    if (!preset_config) {
      LOG(ERROR) << "Error importing preset patterns '" << preset << "'.";
      return;
    }
    pattern_map = preset_config->GetMap("recognizer/patterns");
    load_patterns(this, pattern_map);
  }
  pattern_map = config->GetMap("recognizer/patterns");
  load_patterns(this, pattern_map);
}

---

calculus.cc 105行

// Transformation

Calculation* Transformation::Parse(const vector<string>& args) {
  if (args.size() < 3)
    return NULL;
  const string& left(args[1]);
  const string& right(args[2]);
  if (left.empty())
    return NULL;
  Transformation* x = new Transformation;
  x->pattern_.assign(left); //这里有内存泄漏
  x->replacement_.assign(right);
  return x;
}

[lotem]

calculus.cc 註釋出來那句是 boost::regex::assign(const std::string&, ...)
我認爲內存泄露在於把字符串編譯成正則表達式會發生異常，屆時 x 這個指針所指向的對象就泄露了。

recognizer.cc 這一處我沒有完全懂，但是 RecognizerPatterns::mapped_type 也就是 (*patterns)[...] 的類型也是 boost::regex。猜測泄露的可能是 map 分配的內存。

正則表達式正確的情況下應該是無虞的。看來應該先編譯好正則表達式處理異常再賦值。
x 變量應當改用 the<T> = unique_ptr<T>。