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

Reply via email to