Changes between Initial Version and Version 1 of Ticket #2669, comment 14


Ignore:
Timestamp:
Jan 18, 2015, 10:58:20 AM (9 years ago)
Author:
historic_bruno

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #2669, comment 14

    initial v1  
    22> We have already figured out this one yesterday (const correctness problem when copying) and actually also one of those you fixed. Sorry, I should have told you about that. Anyway, I'll attach the patch here. I've not removed const in my solution but instead implemented a workaround for the clang bug that causes this error. I haven't committed it yet because the workaround needs C++11 and should be added when C++11 gets enabled.
    33
    4 Is that really a clang bug though? I notice there hasn't been any update or progress on the ticket, and I was reading a bit on C++11 which made it seem (to me) that this could be explained by some interaction of the typecast operator constness and implicit copy/move operators. I also wasn't convinced that returning a const value made any sense there, so it semeed harmless to change.
     4Is that really a clang bug though? I notice there hasn't been any update or progress on the [http://llvm.org/bugs/show_bug.cgi?id=16042 ticket], and I was reading a bit on C++11 which made it seem (to me) that this could be explained by some interaction of the typecast operator constness and implicit copy/move operators. I also wasn't convinced that returning a const value made any sense there, so it semeed harmless to change.