jarro2783 / cxxopts

Lightweight C++ command line option parser
MIT License
4.22k stars 592 forks source link

Add missing `#include<locale>` #368

Closed hillmich closed 2 years ago

hillmich commented 2 years ago

The recent change in #364 used std::isblank(charT, locale) from the header locale but did not include this header. Instead it included cctype which only provides std::isblank(int). This caused a break for Xcode (see discussion in #364).

This tiny PR adds the required #include<locale>. (It might be worthwhile to switch all functions from cctype to locale.)


This change is Reviewable

jarro2783 commented 2 years ago

Well spotted, thanks.