Ocena biblioteki statycznej do parsowania CMD

Witajcie,

Pisałem tę bibliotekę dawno. Ogólnie rzecz biorąc to był chyba pierwszy większy i o praktycznym zastosowaniu kod. Jeśli ktoś byłby zainteresowany to prosiłbym o ocenę kodu (szczególnie chodzi mi czy nie ma jakichś złych nawyków itd.). Na licencjach się nie znam. Chyba MIT może być do opensource, a nie tylko do binarek? Najbardziej odpowiadał właśnie MIT (to biblioteka statyczna więc gdyby była na GNU programy jej używające musiały by być także na GNU). Bibliotekę kompilowałem przez załączone Makefile i testowałem na GNU/Linux (da się pewnie to skompilować na Win bo właśnie tam była pierwotnie pisana, ponieważ używa tylko standardowych bibliotek C++). Link do GitHub: https://github.com/patrykbajos/CmdParser

Nie wczytałem się w logikę, więc tylko rzeczy które się rzucają w oczy:

1. Używanie nazw zaczynających się od _, np. _name
2. Przekazywanie obiektów przez wartość
3. Niepotrzebne używanie this->

Zamiast:

if (it == this->flags.end())
{
    return false;
}
else
{
    return true;
}

można

return it != flags.end();