ahmadsamir added inline comments.

INLINE COMMENTS

> kdirmodel.cpp:222
> +        QUrl 
> parent(url.adjusted(QUrl::RemoveFilename|QUrl::StripTrailingSlash));
> +        if (url.path() == QLatin1String("/"))
> +            parent.setPath(QString());

Braces around if.

> kdirmodel.h:79
> +     * @param url   the URL of the directory whose contents should be listed.
> +     *              Unless ShowRoot is set, the item for this directory will 
> NOT be shown, the model starts at its children.
> +     * @param flags see OpenUrlFlag

I suggest:
s/its children/the first child/ OR
s/at its children/directly at the first child/

REPOSITORY
  R241 KIO

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

To: dfaure, stefanocrocco, elvisangelaccio, meven, apol
Cc: ahmadsamir, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, 
ngraham, bruns

Reply via email to