Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upTypo in 37f17f3808c1c97bd72c165a530c67a4bfb82edb? #45
Comments
This comment has been minimized.
This comment has been minimized.
|
You are entirely righ. |
added a commit
that referenced
this issue
May 6, 2015
This comment has been minimized.
This comment has been minimized.
|
Fixed in 6247e58 |
annejan
closed this
May 6, 2015
added a commit
that referenced
this issue
Jun 1, 2016
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
rdoeffinger commentedMay 6, 2015
That commit changes
output.replace(QRegExp("\n"), "
");
to
output.replace(QRegExp("\n )"), "
");
was that intentional?
I thought the point was to replace all newlines with HTML tags (which I admit I do not know if it is necessary).
The new code triggers only on a sequence I'd think would almost never appear in the output...