<div dir="ltr"><div>Time to move Modem Manager Qt now?  Last Plasma 5.2.2 is tarred and KF 5.9 is due for tagging in 2 weeks.<br><br></div>Jonathan<br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On 16 March 2015 at 15:44, Jan Grulich <span dir="ltr"><<a href="mailto:jgrulich@redhat.com" target="_blank">jgrulich@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thursday 12 of March 2015 14:12 David Edmundson wrote:<br>
> Looks good to me.<br>
> 2 minor comments.<br>
><br>
> All classes are namespaced, but generictypes.h is not.<br>
> Given these names could easily clash with something else and are publicly<br>
> included, it might be worth putting them in the same namespace.<br>
<br>
</span>Done.<br>
<span class=""><br>
> ModemManager::ModemMessaging::messages can be const?<br>
<br>
</span>Yes, fixed.<br>
<br>
><br>
> David<br>
><br>
> ​<br>
<br>
On Friday 13 of March 2015 00:06 Albert Astals Cid wrote:<br>
> Kill the framework branch?<br>
<br>
Removed.<br>
<span class=""><br>
> Move macros.h and mmdebug.h to _p.h?<br>
<br>
</span>Done.<br>
<span class=""><br>
> Can listBearers and findBearer be const?<br>
<br>
</span>Yep, done.<br>
<span class=""><br>
> In InterfaceType i'd say you can let the enums be there even if the<br>
> MM_CHECK_VERSION doesn't match, makes sure in case some others are added<br>
> later they always have the same "int" value<br>
<br>
</span>Removed check.<br>
<span class=""><br>
> Make BearerStruct, IpConfig and NetworkTimeZonea and the structs in<br>
> generictypes.h classes with dptr in case you ever need more fields in them?<br>
<br>
</span>I made only BearerStruct, IpConfig and NetworkTimeZone as classes. Other<br>
structures are just pairs and won't need more fields in future.<br>
<span class=""><br>
> Add const & to params of ip4ConfigChanged, ip6ConfigChanged and<br>
> networkTimeZoneChanged?<br>
<br>
</span>Done.<br>
<br>
> Cheers,<br>
>   Albert<br>
><br>
<br>
Thanks, is there anything else?<br>
<span class="im HOEnZb"><br>
Regards,<br>
Jan<br>
--<br>
Jan Grulich<br>
Red Hat Czech, s.r.o<br>
<a href="mailto:jgrulich@redhat.com">jgrulich@redhat.com</a><br>
</span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Kde-frameworks-devel mailing list<br>
<a href="mailto:Kde-frameworks-devel@kde.org">Kde-frameworks-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kde-frameworks-devel" target="_blank">https://mail.kde.org/mailman/listinfo/kde-frameworks-devel</a><br>
</div></div></blockquote></div><br></div>