[Github-comments] [geany/geany] error compiling at scintilla with C++ (#2731)

[elextr]

The errors posted in the OP (assuming Scintilla 3.7.5 or earlier, the line is blank in later versions) do indeed look like a known bug in early C++11 compilers that didn't detect the return value was an rvalue and required `std::move` on them.  Of course then once that was fixed, later compilers complained that the explicit `std::move` prevented them from eliding the copy, so the `std::move` had to be removed.

Basically its simply not possible to support both the erroneous compiler and correct ones.  I remember discussing with Neil the issue of needing a compiler upgrade across the GCC 4.8-5 boundary and its ABI break would have on C++ projects that used any pre-compiled C++ as well as Scintilla, but I can't find it again now (paging @codebrainz search services).  But as I said above thats not a problem for Geany which compiles all the C++ it uses and only uses C libraries that are precompiled, and the C abi hasn't been broken since forever.

-- 
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/issues/2731#issuecomment-764058569
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20210120/f47d71dc/attachment.htm>