henricj / dunelegacy

GNU General Public License v2.0
27 stars 5 forks source link

Bad use after move #8

Closed boris768 closed 2 years ago

boris768 commented 2 years ago

https://github.com/henricj/dunelegacy/blob/4cc40be32d7229e66e5056249615392fbfcc75dd/src/CommandManager.cpp#L86-L87 https://github.com/henricj/dunelegacy/blob/4cc40be32d7229e66e5056249615392fbfcc75dd/src/FileClasses/POFile.cpp#L108-L109

dont use variables afrer std::move

henricj commented 2 years ago

Thanks.

henricj commented 2 years ago

...and I just broke things with that "fix".