summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--slicer/db/sqlBinder.h1
-rw-r--r--slicer/db/sqlInsertSerializer.cpp1
-rw-r--r--slicer/db/sqlSelectDeserializer.cpp1
-rw-r--r--slicer/db/sqlSource.h1
-rw-r--r--slicer/db/sqlTablePatchSerializer.cpp1
-rw-r--r--slicer/db/sqlUpdateSerializer.cpp1
-rw-r--r--slicer/db/testConversions.cpp2
-rw-r--r--slicer/db/testInsert.cpp1
-rw-r--r--slicer/db/testMockCommon.cpp1
-rw-r--r--slicer/db/testSelect.cpp1
-rw-r--r--slicer/db/testUpdate.cpp1
-rw-r--r--slicer/ice/serializer.cpp1
-rw-r--r--slicer/json/serializer.cpp2
-rw-r--r--slicer/json/serializer.h1
-rw-r--r--slicer/slicer/any_ptr.h8
-rw-r--r--slicer/slicer/enumMap.cpp1
-rw-r--r--slicer/slicer/hook-test.cpp1
-rw-r--r--slicer/slicer/metadata-test.cpp2
-rw-r--r--slicer/slicer/metadata.cpp1
-rw-r--r--slicer/slicer/modelParts.h1
-rw-r--r--slicer/slicer/modelPartsTraits.cpp1
-rw-r--r--slicer/slicer/modelPartsTraits.h1
-rw-r--r--slicer/slicer/modelPartsTypes.impl.cpp1
-rw-r--r--slicer/slicer/slicer.cpp3
-rw-r--r--slicer/slicer/slicer.h1
-rw-r--r--slicer/test/compilation.cpp21
-rw-r--r--slicer/test/conversions.cpp2
-rw-r--r--slicer/test/conversions.h1
-rw-r--r--slicer/test/preprocessor.cpp2
-rw-r--r--slicer/test/serializers.cpp3
-rw-r--r--slicer/tool/icemetadata.cpp7
-rw-r--r--slicer/tool/icemetadata.h5
-rw-r--r--slicer/xml/serializer.cpp1
-rw-r--r--slicer/xml/serializer.h1
-rw-r--r--slicer/xml/testSpecifics.cpp1
35 files changed, 22 insertions, 59 deletions
diff --git a/slicer/db/sqlBinder.h b/slicer/db/sqlBinder.h
index 4453d4d..e4e00ca 100644
--- a/slicer/db/sqlBinder.h
+++ b/slicer/db/sqlBinder.h
@@ -2,7 +2,6 @@
#include <Ice/Config.h>
#include <boost/date_time/posix_time/posix_time.hpp>
-#include <memory>
#include <slicer/modelParts.h>
#include <string>
diff --git a/slicer/db/sqlInsertSerializer.cpp b/slicer/db/sqlInsertSerializer.cpp
index 0881a1b..6119f54 100644
--- a/slicer/db/sqlInsertSerializer.cpp
+++ b/slicer/db/sqlInsertSerializer.cpp
@@ -8,7 +8,6 @@
#include <command_fwd.h>
#include <compileTimeFormatter.h>
#include <connection.h>
-#include <functional>
#include <memory>
#include <modifycommand.h>
#include <slicer/modelParts.h>
diff --git a/slicer/db/sqlSelectDeserializer.cpp b/slicer/db/sqlSelectDeserializer.cpp
index f23055e..f1c2726 100644
--- a/slicer/db/sqlSelectDeserializer.cpp
+++ b/slicer/db/sqlSelectDeserializer.cpp
@@ -2,7 +2,6 @@
#include "sqlExceptions.h"
#include "sqlSource.h"
#include <column.h>
-#include <memory>
#include <selectcommand.h>
#include <slicer/common.h>
#include <slicer/hookMap.h>
diff --git a/slicer/db/sqlSource.h b/slicer/db/sqlSource.h
index 84e88d4..2479c2e 100644
--- a/slicer/db/sqlSource.h
+++ b/slicer/db/sqlSource.h
@@ -2,7 +2,6 @@
#include "slicer/modelParts.h"
#include <Ice/Config.h>
-#include <memory>
#include <string>
namespace DB {
diff --git a/slicer/db/sqlTablePatchSerializer.cpp b/slicer/db/sqlTablePatchSerializer.cpp
index ec0e344..f7d8fa9 100644
--- a/slicer/db/sqlTablePatchSerializer.cpp
+++ b/slicer/db/sqlTablePatchSerializer.cpp
@@ -3,7 +3,6 @@
#include "sqlInsertSerializer.h"
#include <compileTimeFormatter.h>
#include <connection.h>
-#include <memory>
#include <scopeExit.h>
#include <slicer/modelParts.h>
#include <string>
diff --git a/slicer/db/sqlUpdateSerializer.cpp b/slicer/db/sqlUpdateSerializer.cpp
index 152c923..1cc6dc7 100644
--- a/slicer/db/sqlUpdateSerializer.cpp
+++ b/slicer/db/sqlUpdateSerializer.cpp
@@ -5,7 +5,6 @@
#include <common.h>
#include <compileTimeFormatter.h>
#include <connection.h>
-#include <functional>
#include <memory>
#include <modifycommand.h>
#include <slicer/modelParts.h>
diff --git a/slicer/db/testConversions.cpp b/slicer/db/testConversions.cpp
index f957150..09286db 100644
--- a/slicer/db/testConversions.cpp
+++ b/slicer/db/testConversions.cpp
@@ -1,8 +1,6 @@
#include <Ice/Config.h>
#include <boost/date_time/posix_time/posix_time.hpp>
#include <boost/numeric/conversion/cast.hpp>
-#include <cstdint>
-#include <memory>
#include <testModels.h>
#include <visibility.h>
diff --git a/slicer/db/testInsert.cpp b/slicer/db/testInsert.cpp
index 24cae7e..65d6c40 100644
--- a/slicer/db/testInsert.cpp
+++ b/slicer/db/testInsert.cpp
@@ -15,7 +15,6 @@
#include <memory>
#include <slicer/slicer.h>
#include <string>
-#include <string_view>
#include <vector>
// IWYU pragma: no_forward_declare Slicer::UnsupportedModelType
diff --git a/slicer/db/testMockCommon.cpp b/slicer/db/testMockCommon.cpp
index 181a970..f8af309 100644
--- a/slicer/db/testMockCommon.cpp
+++ b/slicer/db/testMockCommon.cpp
@@ -2,7 +2,6 @@
#include <connection_fwd.h>
#include <definedDirs.h>
#include <filesystem>
-#include <memory>
#include <mockDatabase.h>
#include <pq-mock.h>
diff --git a/slicer/db/testSelect.cpp b/slicer/db/testSelect.cpp
index 769e31b..2f73d70 100644
--- a/slicer/db/testSelect.cpp
+++ b/slicer/db/testSelect.cpp
@@ -17,7 +17,6 @@
#include <connection.h>
#include <memory>
#include <string>
-#include <string_view>
#include <vector>
// IWYU pragma: no_forward_declare Slicer::NoRowsReturned
// IWYU pragma: no_forward_declare Slicer::TooManyRowsReturned
diff --git a/slicer/db/testUpdate.cpp b/slicer/db/testUpdate.cpp
index e6368e4..508f0a2 100644
--- a/slicer/db/testUpdate.cpp
+++ b/slicer/db/testUpdate.cpp
@@ -15,7 +15,6 @@
#include <connection.h>
#include <memory>
#include <string>
-#include <string_view>
#include <vector>
// IWYU pragma: no_forward_declare Slicer::NoRowsFound
// IWYU pragma: no_forward_declare Slicer::UnsupportedModelType
diff --git a/slicer/ice/serializer.cpp b/slicer/ice/serializer.cpp
index eb27729..e7f497d 100644
--- a/slicer/ice/serializer.cpp
+++ b/slicer/ice/serializer.cpp
@@ -6,7 +6,6 @@
#include <factory.h>
#include <istream>
#include <iterator>
-#include <memory>
#include <slicer/modelParts.h>
#include <slicer/serializer.h>
diff --git a/slicer/json/serializer.cpp b/slicer/json/serializer.cpp
index 4070231..ab7471e 100644
--- a/slicer/json/serializer.cpp
+++ b/slicer/json/serializer.cpp
@@ -7,14 +7,12 @@
#include <glibmm/ustring.h>
#include <jsonpp.h>
#include <map>
-#include <memory>
#include <optional>
#include <slicer/modelParts.h>
#include <slicer/serializer.h>
#include <string_view>
#include <utility>
#include <variant>
-#include <vector>
NAMEDFACTORY(".js", Slicer::JsonFileSerializer, Slicer::FileSerializerFactory)
NAMEDFACTORY(".js", Slicer::JsonFileDeserializer, Slicer::FileDeserializerFactory)
diff --git a/slicer/json/serializer.h b/slicer/json/serializer.h
index 2e54a2d..ab1995e 100644
--- a/slicer/json/serializer.h
+++ b/slicer/json/serializer.h
@@ -6,7 +6,6 @@
#include <jsonpp.h>
#include <slicer/modelParts.h>
#include <slicer/serializer.h>
-#include <string>
#include <visibility.h>
namespace Slicer {
diff --git a/slicer/slicer/any_ptr.h b/slicer/slicer/any_ptr.h
index 14199c0..662023a 100644
--- a/slicer/slicer/any_ptr.h
+++ b/slicer/slicer/any_ptr.h
@@ -1,8 +1,5 @@
#pragma once
-#include <memory>
-#include <vector>
-
namespace Slicer {
template<typename T> struct any_ptr {
// cppcheck-suppress noExplicitConstructor
@@ -14,7 +11,7 @@ namespace Slicer {
inline constexpr any_ptr(T & p) noexcept : ptr {&p} { }
// cppcheck-suppress noExplicitConstructor
- // NOLINTNEXTLINE(hicpp-explicit-conversions)
+ // NOLINTNEXTLINE(hicpp-explicit-conversions,cppcoreguidelines-rvalue-reference-param-not-moved)
inline constexpr any_ptr(T && p) noexcept : ptr {&p} { }
template<typename S>
@@ -43,7 +40,8 @@ namespace Slicer {
return *ptr;
}
- inline constexpr explicit operator bool() const noexcept
+ inline constexpr explicit
+ operator bool() const noexcept
{
return ptr;
}
diff --git a/slicer/slicer/enumMap.cpp b/slicer/slicer/enumMap.cpp
deleted file mode 100644
index 8d6b96a..0000000
--- a/slicer/slicer/enumMap.cpp
+++ /dev/null
@@ -1 +0,0 @@
-#include "enumMap.h"
diff --git a/slicer/slicer/hook-test.cpp b/slicer/slicer/hook-test.cpp
index 2fa1b8b..ac9fe3c 100644
--- a/slicer/slicer/hook-test.cpp
+++ b/slicer/slicer/hook-test.cpp
@@ -1,7 +1,6 @@
#include "hookMap.h"
#include "modelPartsTypes.h"
#include "modelPartsTypes.impl.h"
-#include <array>
#include <iterator>
#include <string>
#include <string_view>
diff --git a/slicer/slicer/metadata-test.cpp b/slicer/slicer/metadata-test.cpp
index f6263f1..a72b844 100644
--- a/slicer/slicer/metadata-test.cpp
+++ b/slicer/slicer/metadata-test.cpp
@@ -1,9 +1,7 @@
#include "metadata.h"
#include <array>
#include <optional>
-#include <string>
#include <string_view>
-#include <utility>
namespace test {
constexpr auto rc = [](std::string_view sv) {
diff --git a/slicer/slicer/metadata.cpp b/slicer/slicer/metadata.cpp
deleted file mode 100644
index f5a1b66..0000000
--- a/slicer/slicer/metadata.cpp
+++ /dev/null
@@ -1 +0,0 @@
-#include "metadata.h"
diff --git a/slicer/slicer/modelParts.h b/slicer/slicer/modelParts.h
index c9be22e..d6e58b1 100644
--- a/slicer/slicer/modelParts.h
+++ b/slicer/slicer/modelParts.h
@@ -5,7 +5,6 @@
#include <Ice/Config.h>
#include <c++11Helpers.h>
#include <functional>
-#include <memory>
#include <optional>
#include <string>
#include <string_view>
diff --git a/slicer/slicer/modelPartsTraits.cpp b/slicer/slicer/modelPartsTraits.cpp
deleted file mode 100644
index ca770cd..0000000
--- a/slicer/slicer/modelPartsTraits.cpp
+++ /dev/null
@@ -1 +0,0 @@
-#include "modelPartsTraits.h"
diff --git a/slicer/slicer/modelPartsTraits.h b/slicer/slicer/modelPartsTraits.h
index 7e2daa2..cca1899 100644
--- a/slicer/slicer/modelPartsTraits.h
+++ b/slicer/slicer/modelPartsTraits.h
@@ -5,7 +5,6 @@
#include <functional>
#include <tuple>
#include <type_traits>
-#include <utility>
namespace Slicer {
// Function traits helpers
diff --git a/slicer/slicer/modelPartsTypes.impl.cpp b/slicer/slicer/modelPartsTypes.impl.cpp
deleted file mode 100644
index 639f718..0000000
--- a/slicer/slicer/modelPartsTypes.impl.cpp
+++ /dev/null
@@ -1 +0,0 @@
-#include "modelPartsTypes.impl.h"
diff --git a/slicer/slicer/slicer.cpp b/slicer/slicer/slicer.cpp
index 5a253b0..a912077 100644
--- a/slicer/slicer/slicer.cpp
+++ b/slicer/slicer/slicer.cpp
@@ -1,8 +1,5 @@
-#include "slicer.h"
-#include "slicer/modelParts.h"
#include <common.h>
#include <compileTimeFormatter.h>
-#include <utility>
namespace Slicer {
AdHocFormatter(InvalidEnumerationSymbolMsg, "Invalid enumeration symbol [%?] for type [%?]");
diff --git a/slicer/slicer/slicer.h b/slicer/slicer/slicer.h
index a4b43f3..d57214d 100644
--- a/slicer/slicer/slicer.h
+++ b/slicer/slicer/slicer.h
@@ -1,6 +1,5 @@
#pragma once
-#include <memory>
#include <slicer/modelParts.h>
#include <slicer/serializer.h>
diff --git a/slicer/test/compilation.cpp b/slicer/test/compilation.cpp
index 3cdfca3..8521643 100644
--- a/slicer/test/compilation.cpp
+++ b/slicer/test/compilation.cpp
@@ -11,7 +11,6 @@
#include "structs.h"
#include <Ice/Config.h>
#include <memory>
-#include <optional>
#include <string>
#include <typeinfo>
#include <vector>
@@ -52,15 +51,17 @@ BOOST_AUTO_TEST_CASE(compile_auto_modelpart_type_class)
ModelPartType::Complex, { BOOST_CHECK_THROW(mpp->OnContained(DontCall), std::logic_error); });
}
-static void
-hookHandler(std::vector<std::string> * names, const std::string & name, Slicer::ModelPartParam mpp,
- const Slicer::HookCommon * h)
-{
- names->push_back(name);
- BOOST_REQUIRE(mpp);
- BOOST_CHECK_THROW(mpp->OnContained(DontCall), std::logic_error);
- BOOST_REQUIRE(h);
- BOOST_REQUIRE_EQUAL(h->name, name);
+namespace {
+ void
+ hookHandler(std::vector<std::string> * names, const std::string & name, Slicer::ModelPartParam mpp,
+ const Slicer::HookCommon * h)
+ {
+ names->push_back(name);
+ BOOST_REQUIRE(mpp);
+ BOOST_CHECK_THROW(mpp->OnContained(DontCall), std::logic_error);
+ BOOST_REQUIRE(h);
+ BOOST_REQUIRE_EQUAL(h->name, name);
+ }
}
BOOST_AUTO_TEST_CASE(compile_auto_modelpart_type_sequenceclasses)
diff --git a/slicer/test/conversions.cpp b/slicer/test/conversions.cpp
index 193cd4f..221ce6f 100644
--- a/slicer/test/conversions.cpp
+++ b/slicer/test/conversions.cpp
@@ -3,10 +3,8 @@
#include <boost/date_time/gregorian_calendar.hpp>
#include <boost/date_time/time.hpp>
#include <boost/numeric/conversion/cast.hpp>
-#include <cstdint>
#include <cstdio>
#include <ctime>
-#include <memory>
#include <slicer/modelParts.h>
#include <slicer/modelPartsTypes.h>
#include <stdexcept>
diff --git a/slicer/test/conversions.h b/slicer/test/conversions.h
index f503f97..ff4af3e 100644
--- a/slicer/test/conversions.h
+++ b/slicer/test/conversions.h
@@ -5,7 +5,6 @@
#include <Ice/Config.h>
#include <Ice/Optional.h>
#include <boost/date_time/posix_time/posix_time.hpp>
-#include <optional>
#include <slicer/modelPartsTypes.h>
#include <string>
#include <visibility.h>
diff --git a/slicer/test/preprocessor.cpp b/slicer/test/preprocessor.cpp
index e85a42a..fea951b 100644
--- a/slicer/test/preprocessor.cpp
+++ b/slicer/test/preprocessor.cpp
@@ -6,10 +6,8 @@
#include <cstdio>
#include <definedDirs.h>
#include <filesystem>
-#include <map>
#include <numeric>
#include <span>
-#include <string>
#include <utility>
#include <vector>
diff --git a/slicer/test/serializers.cpp b/slicer/test/serializers.cpp
index ae9a30e..1780237 100644
--- a/slicer/test/serializers.cpp
+++ b/slicer/test/serializers.cpp
@@ -684,7 +684,8 @@ BOOST_AUTO_TEST_CASE(typeid_specifies_same)
auto d1 = Slicer::DeserializeAny<Slicer::XmlStreamDeserializer, TestModule::D1Ptr>(in);
BOOST_REQUIRE(d1);
BOOST_CHECK_EQUAL(d1->a, 3);
- BOOST_CHECK_EQUAL(typeid(*d1).name(), typeid(TestModule::D1).name());
+ auto d1i = *d1;
+ BOOST_CHECK_EQUAL(typeid(d1i).name(), typeid(TestModule::D1).name());
}
BOOST_DATA_TEST_CASE(typeid_specifies_bad,
diff --git a/slicer/tool/icemetadata.cpp b/slicer/tool/icemetadata.cpp
index 4804542..3dfbee9 100644
--- a/slicer/tool/icemetadata.cpp
+++ b/slicer/tool/icemetadata.cpp
@@ -5,12 +5,13 @@
// IWYU pragma: no_include <list>
namespace Slicer {
- IceMetaData::IceMetaData(Slice::StringList && as)
+ IceMetaData::IceMetaData(Slice::StringList as)
{
arr.reserve(as.size());
- std::for_each(as.begin(), as.end(), [this](auto && a) {
- auto & md = arr.emplace_back(a, std::string_view {});
+ std::transform(as.begin(), as.end(), std::back_inserter(arr), [](auto && a) {
+ decltype(arr)::value_type md {std::forward<decltype(a)>(a), std::string_view {}};
md.second = std::string_view(md.first).substr(0, md.first.rfind(':'));
+ return md;
});
_begin = arr.begin();
_end = arr.end();
diff --git a/slicer/tool/icemetadata.h b/slicer/tool/icemetadata.h
index 87d34f2..89dc422 100644
--- a/slicer/tool/icemetadata.h
+++ b/slicer/tool/icemetadata.h
@@ -6,7 +6,6 @@
#include <slicer/metadata.h>
#include <string>
#include <string_view>
-#include <utility>
#include <vector>
#include <visibility.h>
@@ -16,10 +15,10 @@ namespace Slicer {
static constexpr std::string_view slicer_prefix {"slicer"};
explicit IceMetaData() = default;
- explicit IceMetaData(Slice::StringList && arr);
+ explicit IceMetaData(Slice::StringList arr);
~IceMetaData() = default;
- SPECIAL_MEMBERS_DELETE(IceMetaData);
+ SPECIAL_MEMBERS_DEFAULT(IceMetaData);
[[nodiscard]] static std::vector<std::string_view> split(std::string_view metaData);
diff --git a/slicer/xml/serializer.cpp b/slicer/xml/serializer.cpp
index 8e351b6..6c0fc27 100644
--- a/slicer/xml/serializer.cpp
+++ b/slicer/xml/serializer.cpp
@@ -21,7 +21,6 @@
#include <factory.h>
#include <lazyPointer.h>
#include <list>
-#include <memory>
#include <optional>
#include <slicer/metadata.h>
#include <slicer/modelParts.h>
diff --git a/slicer/xml/serializer.h b/slicer/xml/serializer.h
index 6ee0bff..9f75d2c 100644
--- a/slicer/xml/serializer.h
+++ b/slicer/xml/serializer.h
@@ -12,7 +12,6 @@
#include <fstream>
#include <slicer/modelParts.h>
#include <slicer/serializer.h>
-#include <string>
#include <visibility.h>
namespace Glib {
diff --git a/slicer/xml/testSpecifics.cpp b/slicer/xml/testSpecifics.cpp
index 036c03d..37199a2 100644
--- a/slicer/xml/testSpecifics.cpp
+++ b/slicer/xml/testSpecifics.cpp
@@ -15,7 +15,6 @@
#pragma GCC diagnostic pop
#include <slicer/serializer.h>
#include <slicer/slicer.h>
-#include <typeinfo>
#include <xmlExceptions.h>
// IWYU pragma: no_forward_declare Slicer::BadBooleanValue