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