elvisangelaccio accepted this revision.
elvisangelaccio added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> job_error.cpp:30
>  
> +static const int maxFilePathLength = 80;
> +

Maybe it's worth using the `s_` prefix here.

REPOSITORY
  R241 KIO

BRANCH
  squeeze-very-long-filenames (branched from master)

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

To: ngraham, #frameworks, #dolphin, elvisangelaccio
Cc: elvisangelaccio, kde-frameworks-devel, michaelh, ngraham, bruns

Reply via email to