You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2016/09/20 16:13:20 UTC
[jira] [Commented] (LANG-1023) WordUtils.wrap: Customizable
breakable characters (not just space/whitespace)
[ https://issues.apache.org/jira/browse/LANG-1023?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15506989#comment-15506989 ]
ASF GitHub Bot commented on LANG-1023:
--------------------------------------
Github user britter commented on a diff in the pull request:
https://github.com/apache/commons-lang/pull/179#discussion_r79649084
--- 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 --
The table is a column for the `wrapOn` parameter
> 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)