Review Request: Separate include pathes with newlines when listing them in a missing include file assistant description.

Ivan Shapovalov intelfx100 at gmail.com
Sat Jan 5 18:05:41 UTC 2013


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

(Updated Jan. 5, 2013, 6:05 p.m.)


Review request for KDevelop.


Changes
-------

Used white-space:pre-wrap instead of "\n" -> "<br/>" by Milian's suggestion.


Description (updated)
-------

Qt::escape() doesn't handle newline characters. Fix this by adding white-space:pre-wrap style to the problem description in the navigation context.


Diffs (updated)
-----

  language/duchain/navigation/problemnavigationcontext.cpp a705ded 

Diff: http://git.reviewboard.kde.org/r/108148/diff/


Testing
-------

Apparently works.


Thanks,

Ivan Shapovalov

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20130105/e1240787/attachment.html>


More information about the KDevelop-devel mailing list