{"thread":{"id":"18860","subject":"[EGIT PATCH] Do not fill paragraphs in the commit message viewer after any sob.","startedAt":"2009-04-13T23:26:10Z","lastAt":"2009-04-13T23:26:10Z","messageCount":1,"participants":["Robin Rosenberg"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"111252","messageId":"1239665170-24090-1-git-send-email-robin.rosenberg@dewire.com","threadId":"18860","inReplyTo":null,"subject":"[EGIT PATCH] Do not fill paragraphs in the commit message viewer after any sob.","fromName":"Robin Rosenberg","fromEmail":"robin.rosenberg@dewire.com","sentAt":"2009-04-13T23:26:10Z","receivedAt":"2009-04-13T23:26:10Z","isPatch":true,"sender":{"key":"robin.rosenberg@dewire.com","avatar":"https://avatars.githubusercontent.com/u/46357?v=4"},"body":"The fill-paragraph rules are ad-hoc and this seems like an improvement.\nSometimes we add strange extra tags after the sob lines, so don't\npretty-format these.\n\nSigned-off-by: Robin Rosenberg <robin.rosenberg@dewire.com>\n---\n .../ui/internal/history/CommitMessageViewer.java   |   12 +++++++++---\n 1 files changed, 9 insertions(+), 3 deletions(-)\n\ndiff --git a/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/history/CommitMessageViewer.java b/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/history/CommitMessageViewer.java\nindex d21555c..fc83782 100644\n--- a/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/history/CommitMessageViewer.java\n+++ b/org.spearce.egit.ui/src/org/spearce/egit/ui/internal/history/CommitMessageViewer.java\n@@ -181,12 +181,18 @@ private void format() {\n \t\tmakeGrayText(d, styles);\n \t\td.append(\"\\n\");\n \t\tString msg = commit.getFullMessage();\n-\t\tif (fill)\n-\t\t\tmsg = msg.replaceAll(\"([\\\\w.; \\t])\\n(\\\\w)\",\"$1 $2\");\n+\t\tPattern p = Pattern.compile(\"\\n([A-Z](?:[A-Za-z]+-)+by: [^\\n]+)\");\n+\t\tif (fill) {\n+\t\t\tMatcher spm = p.matcher(msg);\n+\t\t\tif (spm.find()) {\n+\t\t\t\tString subMsg = msg.substring(0, spm.end());\n+\t\t\t\tmsg = subMsg.replaceAll(\"([\\\\w.,; \\t])\\n(\\\\w)\", \"$1 $2\")\n+\t\t\t\t\t\t+ msg.substring(spm.end());\n+\t\t\t}\n+\t\t}\n \t\tint h0 = d.length();\n \t\td.append(msg);\n \n-\t\tPattern p = Pattern.compile(\"\\n([A-Z](?:[A-Za-z]+-)+by: [^\\n]+)\");\n \t\tMatcher matcher = p.matcher(msg);\n \t\twhile (matcher.find()) {\n \t\t\tstyles.add(new StyleRange(h0 + matcher.start(), matcher.end()-matcher.start(), null,  null, SWT.ITALIC));\n-- \n1.6.2.2.446.gfbdc0\n"}]}