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

[ayushjariyal]

Hey @cvvergara,
While running clang-tidy, I noticed that there are still some warnings related to cppcoreguidelines-init-variables.
Would it be okay if I work on fixing those? I’d be happy to contribute improvements there.