dfaure added inline comments. INLINE COMMENTS
> ahmadsamir wrote in kcoredirlister.cpp:1044 > Noted. Actually, I was wrong. I talked to the head of the LLVM/clang development team at Apple and he told me that returning a const value has zero effect whatsoever. I.e. in the version of your patch where you were returning a const value, the range for would *still* call the non-const begin() and end(). Good thing I made you change that :-) His plan is to make a future version of the C++ standard deprecate const arguments and const return values in function declarations. (toplevel const only, const-ref is fine of course). REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D23902 To: ahmadsamir, kde-frameworks-devel, dfaure Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
