> 1. Remove "m_" prefix as they are not private members. Actually it's not only for private members[1], but I think it's not mandatory, anyway it's not objection, and I am OK with that, just share the doc
[1] "When structs and/or classes have member functions, prefer to name data members with a leading m_ and static data members with a leading s_." https://gcc.gnu.org/codingconventions.html
