Review Request: dataprotocol: simplify helper code

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104860/#review13611
-----------------------------------------------------------


This review has been submitted with commit 279694530bfd6df3b8c5a8b31d1350bd2720a81f by Rolf Eike Beer to branch KDE/4.8.

- Commit Hook


On May 6, 2012, 6:10 p.m., Rolf Eike Beer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104860/
> -----------------------------------------------------------
> 
> (Updated May 6, 2012, 6:10 p.m.)
> 
> 
> Review request for kdelibs.
> 
> 
> Description
> -------
> 
> -add some "const" and "static"
> -remove function parameters that always have the same values, use local statics
>  in the function to hold these
> -QChar(QLatin1Char('\0')) => QChar()
> -QChar == QLatin1Char('\0') => QChar::isNull()
> 
> 
> Diffs
> -----
> 
>   kio/kio/dataprotocol.cpp e614476 
> 
> Diff: http://git.reviewboard.kde.org/r/104860/diff/
> 
> 
> Testing
> -------
> 
> -build whole kdelibs
> -dataprotocol testcases still pass
> 
> 
> Thanks,
> 
> Rolf Eike Beer
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120509/e0fe74aa/attachment.htm>