[LyX/master] use std::move() instead of copying when value is no longer used
Jean-Marc Lasgouttes
lasgouttes at lyx.org
Tue Sep 10 13:21:51 UTC 2024
Le 10/09/2024 à 15:01, Pavel Sanda a écrit :
> I find that std::move construct make the readbility worse for me. Not seeing it
> on the left side of the assignment makes it easy to overlook that the variable
> might be actually empty after move and use it in the subsequent code later.
>
> Unless we are optimizing for speed in some critical section I would prefer
> to use the old inefficient way...
I agree with that. However, I have not found the toggle that tells
Coverity to skip this proposal. This is most of our reports.
JMarc
More information about the lyx-devel
mailing list