[
https://issues.apache.org/jira/browse/LANG-1023?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15509591#comment-15509591
]
ASF GitHub Bot commented on LANG-1023:
--------------------------------------
Github user marko-bekhta commented on a diff in the pull request:
https://github.com/apache/commons-lang/pull/179#discussion_r79809527
--- Diff: src/main/java/org/apache/commons/lang3/text/WordUtils.java ---
@@ -172,6 +173,84 @@ public static String wrap(final String str, final int
wrapLength) {
* @return a line with newlines inserted, <code>null</code> if null
input
*/
public static String wrap(final String str, int wrapLength, String
newLineStr, final boolean wrapLongWords) {
+ return wrap(str, wrapLength, newLineStr, wrapLongWords, " ");
+ }
+
+ /**
+ * <p>Wraps a single line of text, identifying words by <code>'
'</code>.</p>
+ *
+ * <p>Leading spaces on a new line are stripped.
+ * Trailing spaces are not stripped.</p>
+ *
+ * <table border="1" summary="Wrap Results">
+ * <tr>
+ * <th>input</th>
+ * <th>wrapLenght</th>
+ * <th>newLineString</th>
+ * <th>wrapLongWords</th>
--- End diff --
@britter Hi, I've made the requested changes. Can you take a look?
> WordUtils.wrap: Customizable breakable characters (not just space/whitespace)
> -----------------------------------------------------------------------------
>
> Key: LANG-1023
> URL: https://issues.apache.org/jira/browse/LANG-1023
> Project: Commons Lang
> Issue Type: Wish
> Components: lang.text.*
> Affects Versions: 3.3.2
> Reporter: Andrew Pennebaker
> Labels: word,wrap,word-wrap,
> Fix For: Review Patch
>
>
> WordUtils.wrap is good at what it does, but I want slightly different
> semantics. I would like the option to set different characters for word
> breaking, not just space or whitespace, but punctuation marks.
> The current implementation wraps like this:
> flammable/inflammable [original string]
> flammable/inflammable [wrap with large widths]
> flammable/inflammable [wrap with wrapLongWords = false]
> flammable/inflam [wrap with wrapLongWords = true]
> mable
> I would like to configure wrap to break "flammable/inflammable" on
> punctuation marks as well as whitespace, so that it wraps like this:
> flammable/ [wrap with additionalBreakCharacters = "/+-"]
> inflammable
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
