bruns added inline comments.

INLINE COMMENTS

> fakecdrom.cpp:48
> +
> +    for (auto it = map.cbegin(); it != map.cend(); ++it) {
> +        if (supported_medialist.contains(it.value())) {

The obviously better solution is to iterate over `supported_medialist` and swap 
the key and value of the map.

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

Reply via email to