| 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 |
| Move to ComparableUtils. </p> |
278 |
| Move to ComparableUtils. </p> |
293 |
| Java 11 Use Optional#isEmpty() |
956 |
| Move to ComparableUtils. </p> |
998 |
| Move to ComparableUtils. </p> |
1068 |
| 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 |
181 |
| org.apache.commons.lang3.reflect.MemberUtils |
Line |
| extract an interface to implement compareParameterSets(...)? |
32 |
| 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(). |
205 |
| org.apache.commons.lang3.time.FastDateParser |
Line |
| Use of getZoneStrings() is discouraged per its Javadoc. |
530 |