Fare9
Fare9
Create test code for all the parts of the parser, this code must contain the asserts to do the checks of the header values in order to know if parsing...
In some applications which are compiled with all the Annotation information, it is necessary to implement these classes in order to avoiding a crash in the application, parsing fails because...
Whenever is possible change the type of some arguments for reference to the object, as if the type is not a reference, argument will be given as copy, and in...
In case of an inheritance chain, make a destructor virtual in order to use correct object destructors. More information can be found [here](https://www.geeksforgeeks.org/virtual-destructor/)
Change all those `for` loops where iterators are being used, for range-`for` loops, check documentation: https://en.cppreference.com/w/cpp/language/range-for
It's not necessary to use `this` inside of a method when another variable with same name does not exist, so the use of `this->` is not necessary.
In case a variable is a constant value, or previously there's always an assignment to that variable, remove unnecessary checks for `null` values or `nullptr` values
Inline functions in C++ header whenever possible, for example some constructors, destructors and simple getters and setters.
Whenever is possible change the constructors for initialization lists. Check this link in order to get more information about it: [constructor](https://en.cppreference.com/w/cpp/language/constructor)