D20737: Convert c arrays to std::array

[Méven Car]

meven added a comment.


  For reference:
  https://en.cppreference.com/w/cpp/container/array
  
  It seems like a reasonable change to me.

INLINE COMMENTS

> kfileitemmodel.cpp:2299
>  {
> -    static const RoleInfoMap rolesInfoMap[] = {
> +    static constexpr std::array<RoleInfoMap, 33> rolesInfoMap{{
>      //  |         role           |        roleType        |                role translation                     |            group translation               | requires Baloo | requires indexer

Too bad we have to specify the length, it would be easy for the compiler to deduce.

REPOSITORY
  R318 Dolphin

REVISION DETAIL
  https://phabricator.kde.org/D20737

To: gawin, #dolphin
Cc: 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/20190426/e52d5c88/attachment.htm>