D29138: [Solid] Replace foreach (deprecated) with range/index for
Stefan BrĂ¼ns
noreply at phabricator.kde.org
Sat Apr 25 18:26:56 BST 2020
bruns added inline comments.
INLINE COMMENTS
> ahmadsamir wrote in fakeopticaldisc.cpp:43
> I was talking about something like Solid::OpticalDrive::MediumType(0); ContentType has Solid::OpticalDisc::NoContent whereas MediumType has nothing similar.
>
> IIUC named casts are preferred over old style casts, so I suggest either we add a MediumType enumerator e.g. NotSupported = 0x00000 or use a static_cast.
Thats not a cast but a constructor, see QFlags documentation
REPOSITORY
R245 Solid
BRANCH
l-foreach (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D29138
To: ahmadsamir, #frameworks, dfaure, apol, meven
Cc: bruns, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200425/4a06305f/attachment.html>
More information about the Kde-frameworks-devel
mailing list