Re: Review Request: konqueror: strip new line character from URL on paste automatically

2012-08-26 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106184/#review18026 --- This review has been submitted with commit 4a6e5ee19b03f1a802d

Re: Review Request: konqueror: strip new line character from URL on paste automatically

2012-08-26 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106184/#review18024 --- This review has been submitted with commit 38fda3bf750d71266a1

Re: Review Request: konqueror: strip new line character from URL on paste automatically

2012-08-26 Thread Martin Koller
> On Aug. 25, 2012, 10:15 p.m., David Faure wrote: > > Looks good. I'm not sure where the need for ParagraphSeparator and > > LineSeparator comes from though, do websites actually use that? I found these two in QChar and thought it might happen that the clipboard contains not \n but one of the

Re: Review Request: konqueror: strip new line character from URL on paste automatically

2012-08-25 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106184/#review18003 --- Ship it! Looks good. I'm not sure where the need for Paragraph

Re: Review Request: konqueror: strip new line character from URL on paste automatically

2012-08-25 Thread Martin Koller
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106184/ --- (Updated Aug. 25, 2012, 7:56 p.m.) Review request for KDE Base Apps and Da

Re: Review Request: konqueror: strip new line character from URL on paste automatically

2012-08-25 Thread Rolf Eike Beer
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106184/#review17997 --- konqueror/src/konqcombo.cpp

Review Request: konqueror: strip new line character from URL on paste automatically

2012-08-25 Thread Martin Koller
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106184/ --- Review request for KDE Base Apps and David Faure. Description --- Thi