org.apache.commons.lang3.AbstractLangTest |
Line |
Do more to make sure memory is not retained, maybe like Log4j checks for it. |
35 |
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 { |
64 |
Implement rather than convert to String return cs.toString().indexOf(searchChar.toString(), start); } |
69 |
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? |
341 |
org.apache.commons.lang3.ObjectUtils |
Line |
Java 11 Use Optional#isEmpty() |
1043 |
org.apache.commons.lang3.RandomUtilsTest |
Line |
should be <max? |
73 |
should be <max? |
80 |
should be <max? |
89 |
should be <max? |
99 |
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)? |
312 |
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); |
205 |
refine API for escaping/unescaping specific HTML versions |
215 |
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 |
617 |
when breaking BC, consider returning input |
640 |
org.apache.commons.lang3.exception.ExceptionUtils |
Line |
Remove in Lang 4 |
49 |
Remove in Lang 4 |
249 |
org.apache.commons.lang3.math.NumberUtilsTest |
Line |
Expected to be fixed in Java 23 assertTrue(isParsableDouble("ï¼ï¼ï¼")); |
1061 |
Expected to be fixed in Java 23 assertTrue(isParsableFloat("ï¼ï¼ï¼")); |
1070 |
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 |
182 |
org.apache.commons.lang3.reflect.MemberUtils |
Line |
extract an interface to implement compareParameterSets(...)? |
34 |
org.apache.commons.lang3.reflect.TypeUtilsTest |
Line |
Is this correct? |
552 |
Is this correct? |
554 |
This test returns true unlike the test above. Is this a bug in this test or in the main code? |
768 |
org.apache.commons.lang3.text.StrBuilder |
Line |
make private? |
275 |
make private? |
278 |
org.apache.commons.lang3.text.translate.NumericEntityUnescaper |
Line |
?: Create an OptionsSet class to hide some of the conditional logic below |
66 |
org.apache.commons.lang3.text.translate.UnicodeEscaper |
Line |
Handle potential + sign per various Unicode escape implementations |
127 |
org.apache.commons.lang3.time.AbstractFormatCache |
Line |
Before making public move from getDateTimeInstance(Integer, ...) to int; or some other approach. |
37 |
org.apache.commons.lang3.time.DurationFormatUtils |
Line |
Compare performance to see if anything was lost by losing this optimization. |
536 |
Need to handle escaping of ' |
698 |
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. |
530 |