[Github-comments] [geany/geany] Update C++ compiler check to require C++17 (#2862)

[Matthew Brush]

Yeah, as @elextr mentioned, the reason for the check is for an early failure with a message explaining the compiler isn't supported.

Another way to do it would be to do a compile-test for only `std::string_view` to also support non-C++17 compilers which might have already supported `std::string_view`, but then that would have to be updated for each new C++17 type/function we use, and especially with newer Scintilla 4/5 coming, probably LOTS of stuff.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2862#issuecomment-900703838
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20210817/41323e72/attachment-0001.htm>