---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112679/#review40192
---
Ship it!
staging/kservice/src/services/ktraderparsetree.cpp
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112679/
---
(Updated Sept. 17, 2013, 3:49 p.m.)
Status
--
This change has been
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112679/#review40234
---
This review has been submitted with commit
On Sept. 12, 2013, 7:39 a.m., David Faure wrote:
staging/kservice/src/services/ktraderparsetree.cpp, line 715
http://git.reviewboard.kde.org/r/112679/diff/1/?file=189053#file189053line715
kdelibs coding style - no space inside (), '{' on the same line.
foreach() would
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112679/
---
(Updated Sept. 14, 2013, 6:17 p.m.)
Review request for KDE Frameworks and
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112679/#review39864
---
staging/kservice/src/services/ktraderparsetree.cpp
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112679/
---
Review request for KDE Frameworks and David Faure.
Description
---
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112679/#review39859
---
2 issues I'd like to get advise for inline.