Clang tidy checks that need fixes

[cvvergara]

clang tidy checks that need fix

• cppcoreguidelines-avoid-c-arrays,
• cppcoreguidelines-avoid-const-or-ref-data-members,
• cppcoreguidelines-avoid-do-while,
• cppcoreguidelines-avoid-magic-numbers,
• cppcoreguidelines-explicit-virtual-functions,
• cppcoreguidelines-macro-usage,
• cppcoreguidelines-narrowing-conversions,
• cppcoreguidelines-no-malloc,
• cppcoreguidelines-non-private-member-variables-in-classes,
• cppcoreguidelines-owning-memory,
• cppcoreguidelines-pro-bounds-array-to-pointer-decay,
• cppcoreguidelines-pro-bounds-pointer-arithmetic,
• cppcoreguidelines-pro-type-const-cast,
• cppcoreguidelines-pro-type-member-init,
• cppcoreguidelines-pro-type-reinterpret-cast,
• cppcoreguidelines-pro-type-union-access,
• cppcoreguidelines-pro-type-vararg,
• cppcoreguidelines-slicing,
• cppcoreguidelines-special-member-functions,
• cppcoreguidelines-use-default-member-init
• cppcoreguidelines-virtual-class-destructor
• cppcoreguidelines-special-member-functions
• cppcoreguidelines-prefer-member-initializer
• cppcoreguidelines-init-variables