<div dir="ltr"><div>Looks good to me.</div><div>2 minor comments.</div><div><br></div>All classes are namespaced, but generictypes.h is not. <div>Given these names could easily clash with something else and are publicly included, it might be worth putting them in the same namespace.</div><div><br></div><div>ModemManager::ModemMessaging::messages can be const?<br></div><div><br></div><div><br></div><div>David<br></div><div><div><br>​</div></div></div>