NOMINMAXの定義をifndefで括る

[berryzplus]

テストコードの#define NOMINMAXが警告を吐いています。
CMake対応で -G=Ninja の調整をしていて気になったのでPR出します。

C:\projects\sakura\tests\unittests\test-int2dec.cpp:4: warning: "NOMINMAX" redefined
 #define NOMINMAX
 
In file included from C:/msys64/mingw64/include/c++/8.2.0/x86_64-w64-mingw32/bits/c++config.h:508,
                 from C:/msys64/mingw64/include/c++/8.2.0/limits:42,
                 from C:/projects/sakura/tests/googletest/googletest/include/gtest/gtest.h:56,
                 from C:\projects\sakura\tests\unittests\test-int2dec.cpp:1:
C:/msys64/mingw64/include/c++/8.2.0/x86_64-w64-mingw32/bits/os_defines.h:45: note: this is the location of the previous definition
 #define NOMINMAX 1
 
[ 76%] Building CXX object unittests/CMakeFiles/tests1.dir/test-is_mailaddress.cpp.obj

本件、#592 単体テストコードで NOMINMAX を定義しているのを削除する で話題になってる件です。

「削除しよう。」は違う気がするので、ifndefで括って二重定義を回避します。

closes #592;

と書いておけばマージと同時にissueも閉じられるのかな？

[k-takata]

closes 592;

# が必要です。 closes #592

[berryzplus]

closes 592;

# が必要です。 closes #592

あざっす。直しときました 😄

[berryzplus]

またビルドがコケておる・・・と思ったらHTML Helpでした。

とりあえずRe-Runかけました。

[beru]

問題無いかと思います。

が、複数のcppファイルで個別に書いているのはちょっと気になります。
今後テストのcppファイルが増えていった時にまた個々のcppファイル内に同じ記述を書くというのもなんだか…。

[berryzplus]

レビューありがとうございます。

が、複数のcppファイルで個別に書いているのはちょっと気になります。
今後テストのcppファイルが増えていった時にまた個々のcppファイル内に同じ記述を書くというのもなんだか…。

いまは、気の向いたとき気軽にテストコードを書けるようにできるだけ縛りはつけない感じにしときたいなぁ、と思っています。とりあえず今「警告が気になる」という問題は解決できたので、いつか「似たような定型句が数百か所にある」が問題になったときに残りを考えたらいいのかな、と思っています。