mirror of
https://github.com/subsurface/subsurface.git
synced 2024-11-28 05:00:20 +00:00
d242198c99
Since everything is C++ now, we can use unique_ptr<>s. This makes the code significantly shorter, because we can now use the default move constructor and assignment operators. This has a semantic change when std::move()-ing the divelog: now not the contents of the tables are moved, but the pointers. That is, the moved-from object now has no more tables and must not be used anymore. This made it necessary to replace std::move()s by std::swap()s. In that regard, the old code was in principle broken: it used moved-from objects, which may work but usually doesn't. This commit adds a myriad of .get() function calls where the code expects a C-style pointer. The plan is to remove virtually all of them, when we move free-standing functions into the class it acts on. Or, replace C-style pointers by references where we don't support NULL. Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at> |
||
|---|---|---|
| .. | ||
| CMakeLists.txt | ||
| command.cpp | ||
| command.h | ||
| command_base.cpp | ||
| command_base.h | ||
| command_device.cpp | ||
| command_device.h | ||
| command_divelist.cpp | ||
| command_divelist.h | ||
| command_divesite.cpp | ||
| command_divesite.h | ||
| command_edit.cpp | ||
| command_edit.h | ||
| command_edit_trip.cpp | ||
| command_edit_trip.h | ||
| command_event.cpp | ||
| command_event.h | ||
| command_filter.cpp | ||
| command_filter.h | ||
| command_pictures.cpp | ||
| command_pictures.h | ||