Commit f0a6f718 authored by Simon Spannagel's avatar Simon Spannagel
Browse files

Linter: member initialization, namespace nesting

parent 9f3daf62

src/core/geometry/DetectorModel.hpp

+1 −1
Original line number Diff line number Diff line
@@ -570,7 +570,7 @@ namespace allpix { 
                                                   std::move(offset),

                                                   std::move(material),

                                                   std::move(type),

                                                   std::move(location),

                                                   location,

                                                   std::move(full_hole_size),

                                                   std::move(hole_offset)));

        }

src/core/geometry/SupportLayer.hpp

+1 −1
Original line number Diff line number Diff line
@@ -95,7 +95,7 @@ namespace allpix { 
                     ROOT::Math::XYVector hole_offset)

            : size_(std::move(size)), material_(std::move(material)), type_(std::move(type)),

              hole_size_(std::move(hole_size)), offset_(std::move(offset)), hole_offset_(std::move(hole_offset)),

              location_(std::move(location)) {}

              location_(location) {}



        // Actual parameters returned

        ROOT::Math::XYZPoint center_;

src/modules/DepositionGeant4/DepositionGeant4Module.hpp

+1 −1
Original line number Diff line number Diff line
@@ -87,7 +87,7 @@ namespace allpix { 
        GeometryManager* geo_manager_;



        // Pointer to the Geant4 manager (owned by GeometryBuilderGeant4)

        G4RunManager* run_manager_g4_;

        G4RunManager* run_manager_g4_{nullptr};



        virtual void initialize_g4_action();

src/modules/LCIOWriter/LCIOWriterModule.hpp

+1 −1
Original line number Diff line number Diff line
@@ -40,7 +40,7 @@ namespace allpix { 
         */

        LCIOWriterModule(Configuration& config, Messenger* messenger, GeometryManager* geo_manager);



        ~LCIOWriterModule(){};

        ~LCIOWriterModule() = default;



        /**

         * @brief Initialize LCIO and GEAR output files

src/tools/field_parser.h

+17 −23
Original line number Diff line number Diff line
@@ -139,8 +139,7 @@ namespace allpix { 
} // namespace allpix



// Enable versioning for the FieldData class template

namespace cereal {

    namespace detail {

namespace cereal::detail {

    template <class T> struct Version<allpix::FieldData<T>> {

        static const std::uint32_t version;

        static std::uint32_t registerVersion() {
@@ -152,8 +151,7 @@ namespace cereal { 
    };                                          /* end Version */

    template <class T>

    const std::uint32_t Version<allpix::FieldData<T>>::version = Version<allpix::FieldData<T>>::registerVersion();

    } // namespace detail

} // namespace cereal

} // namespace cereal::detail



namespace allpix {
@@ -171,10 +169,8 @@ namespace allpix { 
         * @param quantity Quantity of individual field points, vector (three values per point) or scalar (one value per

         * point)

         */

        explicit FieldParser(const FieldQuantity quantity) {

            // Store quantity: vector or scalar field:

            N_ = static_cast<std::underlying_type<FieldQuantity>::type>(quantity);

        };

        explicit FieldParser(const FieldQuantity quantity)

            : N_(static_cast<std::underlying_type<FieldQuantity>::type>(quantity)){};

        ~FieldParser() = default;



        /**
@@ -399,10 +395,8 @@ namespace allpix { 
         * @param quantity Quantity of individual field points, vector (three values per point) or scalar (one value per

         * point)

         */

        explicit FieldWriter(const FieldQuantity quantity) {

            // Store quantity: vector or scalar field:

            N_ = static_cast<std::underlying_type<FieldQuantity>::type>(quantity);

        };

        explicit FieldWriter(const FieldQuantity quantity)

            : N_(static_cast<std::underlying_type<FieldQuantity>::type>(quantity)){};

        ~FieldWriter() = default;



        /**