astippich added inline comments.

INLINE COMMENTS

> propertydata.cpp:88
> +        if (it.value().isArray()) {
> +            const auto values = it.value().toArray();
> +            for (const auto val : values) {

Can you return arrays as lists here so that current behavior is unchanged, and 
put the behavior change into another patch so that we can merge this diff?

REPOSITORY
  R293 Baloo

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

To: bruns, #baloo, #frameworks, ngraham, poboiko, astippich
Cc: mgallien, kde-frameworks-devel, gennad, domson, ashaposhnikov, michaelh, 
astippich, spoorun, ngraham, bruns, abrahams

Reply via email to