D20737: Convert c arrays to std::array
Pino Toscano
noreply at phabricator.kde.org
Sun May 12 12:21:52 BST 2019
pino added inline comments.
INLINE COMMENTS
> kbaloorolesprovider.cpp:130
>
> for (unsigned int i = 0; i < sizeof(propertyInfoList) / sizeof(PropertyInfo); ++i) {
> m_roleForProperty.insert(propertyInfoList[i].property, propertyInfoList[i].role);
won't the sizeof() here be invalid now?
REVISION DETAIL
https://phabricator.kde.org/D20737
To: gawin, #dolphin, elvisangelaccio
Cc: pino, elvisangelaccio, meven, kfm-devel, #dolphin, alexde, feverfew, spoorun, navarromorales, firef, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20190512/fd525369/attachment.htm>
More information about the kfm-devel
mailing list