[kde-edu]: Review Request: krazy fixes: kdeedu kmplot: single-char QString operations
Pieter Vande Wyngaerde
pietervandewyngaerde+kdereview at gmail.com
Sun Mar 21 19:13:54 CET 2010
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3333/
-----------------------------------------------------------
Review request for KDE-Edu.
Summary
-------
krazy2 Analysis:
Check single-char QString operations for efficiency [doublequote_chars]...OOPS! 11 issues found!
* kmplot/xparser.cpp: line# 272,274,276 (3)
* kmplot/functioneditor.cpp: line# 471,490,491,504 (4)
* kmplot/function.cpp: line# 48 (1)
* kmplot/view.cpp: line# 798,800 (2)
* kmplot/initialconditionseditor.cpp: line# 156 (1)
(this is my first try)
Diffs
-----
trunk/KDE/kdeedu/kmplot/kmplot/function.cpp 1105921
trunk/KDE/kdeedu/kmplot/kmplot/functioneditor.cpp 1105921
trunk/KDE/kdeedu/kmplot/kmplot/initialconditionseditor.cpp 1105921
trunk/KDE/kdeedu/kmplot/kmplot/view.cpp 1105921
trunk/KDE/kdeedu/kmplot/kmplot/xparser.cpp 1105921
Diff: http://reviewboard.kde.org/r/3333/diff
Testing
-------
none
Thanks,
Pieter
More information about the kde-edu
mailing list