D29138: [Solid] Replace foreach (deprecated) with range/index for
    Ahmad Samir 
    noreply at phabricator.kde.org
       
    Fri Apr 24 18:31:44 BST 2020
    
    
  
ahmadsamir added inline comments.
INLINE COMMENTS
> bruns wrote in fakeopticaldisc.cpp:43
> These four lines can be written as
> `content |= map.value(type, Solid::OpticalDisc::ContentType(0));`
Nice. :)
I think Solid::OpticalDisc::NoContent is more readable.
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/20200424/08b6de16/attachment.html>
    
    
More information about the Kde-frameworks-devel
mailing list