| org.apache.commons.lang3.ArraySorter |
Line |
| For 4.0, rename to ArraySort, since we cover the sort() method here, see also ArrayFill. |
25 |
| org.apache.commons.lang3.CharRange |
Line |
| This is no longer public and will be removed later as CharSet is moved to depend on Range. |
32 |
| org.apache.commons.lang3.CharSequenceUtils |
Line |
| Do we assume searchChar is usually relatively small; // If so then calling toString() on it is better than reverting to // the green implementation in the else block return ((String) cs).indexOf((String) searchChar, start); } else { |
61 |
| Implement rather than convert to String return cs.toString().indexOf(searchChar.toString(), start); } |
66 |
| org.apache.commons.lang3.ClassLoaderUtilsTest |
Line |
| How to better test considering this test may be called from an IDE and Maven? |
37 |
| How to better test considering this test may be called from an IDE and Maven? |
43 |
| org.apache.commons.lang3.LocaleUtils |
Line |
| Should this return the default locale? |
331 |
| org.apache.commons.lang3.ObjectUtils |
Line |
| Rename to getIfNull in 4.0 |
583 |
| Java 11 Use Optional#isEmpty() |
1017 |
| org.apache.commons.lang3.RandomUtilsTest |
Line |
| should be <max? |
77 |
| should be <max? |
84 |
| should be <max? |
93 |
| should be <max? |
103 |
| org.apache.commons.lang3.StringEscapeUtils |
Line |
| throw "illegal character: \92" as an Exception if a \ on the end of the Java (as per the compiler)? |
318 |
| org.apache.commons.lang3.StringEscapeUtilsTest |
Line |
| I think this should hold, needs further investigation String unescapedFromEntity = StringEscapeUtils.unescapeHtml4( "𝍢" ); assertEquals( "High Unicode should have been unescaped", original, unescapedFromEntity); |
204 |
| refine API for escaping/unescaping specific HTML versions |
214 |
| org.apache.commons.lang3.Validate |
Line |
| when breaking BC, consider returning value |
94 |
| when breaking BC, consider returning value |
115 |
| when breaking BC, consider returning value |
135 |
| when breaking BC, consider returning value |
156 |
| when breaking BC, consider returning value |
177 |
| when breaking BC, consider returning value |
201 |
| when breaking BC, consider returning value |
273 |
| when breaking BC, consider returning value |
294 |
| when breaking BC, consider returning value |
314 |
| when breaking BC, consider returning value |
335 |
| when breaking BC, consider returning value |
356 |
| when breaking BC, consider returning value |
380 |
| when breaking BC, consider returning type |
402 |
| when breaking BC, consider returning type |
427 |
| when breaking BC, consider returning obj |
449 |
| when breaking BC, consider returning obj |
472 |
| when breaking BC, consider returning input |
591 |
| when breaking BC, consider returning input |
614 |
| org.apache.commons.lang3.exception.ExceptionUtils |
Line |
| Remove in Lang 4 |
48 |
| Remove in Lang 4 |
248 |
| org.apache.commons.lang3.math.NumberUtilsTest |
Line |
| Expected to be fixed in Java 23 assertTrue(isParsableDouble("１２３")); |
1058 |
| Expected to be fixed in Java 23 assertTrue(isParsableFloat("１２３")); |
1067 |
| org.apache.commons.lang3.reflect.FieldUtils |
Line |
| is this workaround still needed? lang requires Java 6 Sun Java 1.3 has a bugged implementation of getField hence we write the code ourselves |
173 |
| org.apache.commons.lang3.reflect.MemberUtils |
Line |
| extract an interface to implement compareParameterSets(...)? |
34 |
| org.apache.commons.lang3.reflect.TypeUtilsTest |
Line |
| Is this correct? |
525 |
| Is this correct? |
527 |
| This test returns true unlike the test above. Is this a bug in this test or in the main code? |
741 |
| org.apache.commons.lang3.text.StrBuilder |
Line |
| make private? |
270 |
| make private? |
273 |
| org.apache.commons.lang3.text.translate.NumericEntityUnescaper |
Line |
| ?: Create an OptionsSet class to hide some of the conditional logic below |
58 |
| org.apache.commons.lang3.text.translate.UnicodeEscaper |
Line |
| Handle potential + sign per various Unicode escape implementations |
128 |
| org.apache.commons.lang3.time.AbstractFormatCache |
Line |
| Before making public move from getDateTimeInstance(Integer, ...) to int; or some other approach. |
38 |
| org.apache.commons.lang3.time.DurationFormatUtils |
Line |
| Compare performance to see if anything was lost by losing this optimization. |
524 |
| Need to handle escaping of ' |
686 |
| org.apache.commons.lang3.time.DurationFormatUtilsTest |
Line |
| Jacoco shows missing coverage for internal negative days |
509 |
| org.apache.commons.lang3.time.DurationUtils |
Line |
| when using Java >= 9: Use TimeUnit.toChronoUnit(). |
162 |
| org.apache.commons.lang3.time.FastDateParser |
Line |
| Use of getZoneStrings() is discouraged per its Javadoc. |
523 |
