bnfcにメモリリーク問題がある

[hangingman]

cpputestでユニットテストをするとmemory leakを指摘される

../test/exp_suite.cpp:123: error: Failure in TEST(exp_suite, testCalc)
    Memory leak(s) found.
Alloc num (52) Leak size: 16 Allocated at: <unknown> and line: 0. Type: "new"
    Memory: <0x55cf60fa8b90> Content:
    0000: c8 ca ef 5f cf 55 00 00  40 84 fa 60 cf 55 00 00 |..._.U..@..`.U..|
Alloc num (51) Leak size: 8 Allocated at: <unknown> and line: 0. Type: "new"
    Memory: <0x55cf60fa8440> Content:
    0000: a8 bd ef 5f cf 55 00 00                          |..._.U..|
Total number of leaks:  2

ワークアラウンド

• ed44b1645b379e8657495dac3e011bb69d60a5f4 再帰的にdeleteを実行している

原因と思われるところ（これは修正済みだったが参考のため残す）

• https://github.com/BNFC/bnfc/issues/347
  • flexの初期化でファイルポインタ(FILE*)ではなく文字列を指定したときのyyrestartの呼び方がおかしい模様
  • https://github.com/BNFC/bnfc/issues/347#issuecomment-802415897 に記載のようにfmemopenを使うことでとりあえず回避できる

workaround

    nask_statements = R"( blah blah blah )";
    std::unique_ptr<Driver> d(new Driver(false, false));
    FILE* stream = fmemopen(&nask_statements, strlen(nask_statements), "r");
    d->Parse<Program>(stream, "test.img");
    fclose(stream);

[hangingman]

https://github.com/CppCodeReviewers/Covariant-Return-Types-and-Smart-Pointers

[hangingman]

振り返るとメモリーリークの原因はすべてC++のclassのメンバ変数をnewしながらも C++のclassのdestructorで開放していないことに起因していた