Make ETL_TYPEDEF Trivially Copyable

[drewr95]

I'm getting a clang-tidy warning when passing around an ETL_TYPEDEF(degrees, float) by copy to pass it as const reference. Doing some research, clang-tidy weighs the check with whether the type is trivially copyable. I think the way the strong types are generated, there is no need to define the copy ctor or assignment operator.

[drewr95]

I see BOOST's strong typedefs are very similar and also not trivially copyable. Is there a reason why? With the way it's implemented I'm not sure why it would need to be defined? Would it be for if the underlying type wasn't trivially copyable?

[jwellbelove]

I've tried a few experiments, with both trivially and non-trivially copyable types, with the default constructor, copy constructor, and assignment operator removed or defaulted, and everything appears to work as expected.

ETL_TYPEDEF(uint32_t, type1_t);
ETL_TYPEDEF(std::vector<uint32_t>, type2_t);

bool b1 = std::is_trivially_copyable_v<type1_t>; // true
bool b2 = std::is_trivially_copyable_v<type2_t>; // false

[drewr95]

I've tried a few experiments, with both trivially and non-trivially copyable types, with the default constructor, copy constructor, and assignment operator removed or defaulted, and everything appears to work as expected.

ETL_TYPEDEF(uint32_t, type1_t); ETL_TYPEDEF(std::vector<uint32_t>, type2_t);

bool b1 = std::is_trivially_copyable_v<type1_t>; // true bool b2 = std::is_trivially_copyable_v<type2_t>; // false

That is awesome news! I'm hoping to have something in within a couple days if you don't beat me to it :)

I'd also aim for the move ctor and assignment to be defaulted as well.