Apache Commons logo Commons Lang

Tag List Report

The following document contains the listing of user tags found in the code. Below is the summary of the occurrences per tag.

Tag Class Total number of occurrences Tag strings used by tag class
Needs Work 40 TODO, FIXME, XXX
Noteable Markers 74 NOTE, NOPMD, NOSONAR

Each tag is detailed below:

Needs Work

Number of occurrences found in the code: 40

org.apache.commons.lang3.CharRange Line
This is no longer public and will be removed later as CharSet is moved to depend on Range. 31
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 { 146
Implement rather than convert to String return cs.toString().indexOf(searchChar.toString(), start); } 151
org.apache.commons.lang3.ObjectUtils Line
Rename to getIfNull in 4.0 599
org.apache.commons.lang3.StringEscapeUtils Line
Create a parent class - 'SinglePassTranslator' ? It would handle the index checking + length returning, and could also have an optimization check method. 250
throw "illegal character: \92" as an Exception if a \ on the end of the Java (as per the compiler)? 289
org.apache.commons.lang3.StringEscapeUtilsTest Line
refine API for escaping/unescaping specific HTML versions 256
I think this should hold, needs further investigation String unescapedFromEntity = StringEscapeUtils.unescapeHtml4( "𝍢" ); assertEquals( "High Unicode should have been unescaped", original, unescapedFromEntity); 489
org.apache.commons.lang3.Validate Line
when breaking BC, consider returning input 839
when breaking BC, consider returning input 863
when breaking BC, consider returning value 971
when breaking BC, consider returning value 996
when breaking BC, consider returning value 1017
when breaking BC, consider returning value 1040
when breaking BC, consider returning value 1061
when breaking BC, consider returning value 1084
when breaking BC, consider returning value 1109
when breaking BC, consider returning value 1134
when breaking BC, consider returning value 1155
when breaking BC, consider returning value 1178
when breaking BC, consider returning value 1199
when breaking BC, consider returning value 1222
when breaking BC, consider returning obj 1248
when breaking BC, consider returning obj 1273
when breaking BC, consider returning type 1299
when breaking BC, consider returning type 1324
org.apache.commons.lang3.exception.ExceptionUtils Line
Remove in Lang 4.0 47
Remove in Lang 4.0 151
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 90
org.apache.commons.lang3.reflect.MemberUtils Line
extract an interface to implement compareParameterSets(...)? 34
Default access superclass workaround. 43
org.apache.commons.lang3.reflect.TypeUtilsTest Line
This test returns true unlike the test above. Is this a bug in this test or in the main code? 837
org.apache.commons.lang3.text.StrBuilder Line
make private? 96
make private? 98
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 115
org.apache.commons.lang3.time.DurationFormatUtils Line
Compare performance to see if anything was lost by losing this optimisation. 286
Need to handle escaping of ' 510
org.apache.commons.lang3.time.DurationUtils Line
when using Java >= 9: Use TimeUnit.toChronoUnit(). 92
org.apache.commons.lang3.time.FormatCache Line
Before making public move from getDateTimeInstance(Integer, ...) to int; or some other approach. 36

Noteable Markers

Number of occurrences found in the code: 74

org.apache.commons.lang3.CharSequenceUtils Line
- we must do a forward traversal for this to avoid duplicating code points 202
org.apache.commons.lang3.CharSet Line
This is no longer public as CharRange is no longer a public class. It may be replaced when CharSet moves to Range. 224
org.apache.commons.lang3.ClassUtils Line
--No comment-- 1080
org.apache.commons.lang3.StringUtils Line
LINT this intentionally uses == to allow for both null 866
LINT this intentionally uses == to allow for both null 959
logic duplicated below START 6738
logic mostly below END 6753
logic mostly duplicated above START 6792
logic duplicated above END 6808
LINT str cannot be null here 8426
org.apache.commons.lang3.builder.EqualsBuilder Line
we cannot store the actual objects in a HashSet, as that would use the very hashCode() we are in the process of calculating. 101
org.apache.commons.lang3.builder.HashCodeBuilder Line
we cannot store the actual objects in a HashSet, as that would use the very hashCode() we are in the process of calculating. 126
This method uses >> and not >>> as Effective Java and Long.hashCode do. Ideally we should switch to >>> at some stage. There are backwards compat issues, so that will have to wait for the time being. cf LANG-342. 803
org.apache.commons.lang3.builder.StandardToStringStyle Line
as this is implementing the abstract class 53
as this is implementing the abstract class 63
as this is implementing the abstract class 76
as this is implementing the abstract class 87
as this is implementing the abstract class 98
as this is implementing the abstract class 108
as this is implementing the abstract class 120
as this is implementing the abstract class 130
as this is implementing the abstract class 143
as this is implementing the abstract class 154
as this is implementing the abstract class 166
as this is implementing the abstract class 176
as this is implementing the abstract class 188
as this is implementing the abstract class 201
as this is implementing the abstract class 213
as this is implementing the abstract class 226
as this is implementing the abstract class 238
as this is implementing the abstract class 251
as this is implementing the abstract class 263
as this is implementing the abstract class 276
as this is implementing the abstract class 288
as this is implementing the abstract class 301
as this is implementing the abstract class 313
as this is implementing the abstract class 326
as this is implementing the abstract class 338
as this is implementing the abstract class 351
as this is implementing the abstract class 365
as this is implementing the abstract class 377
as this is implementing the abstract class 391
as this is implementing the abstract class 403
as this is implementing the abstract class 415
as this is implementing the abstract class 428
as this is implementing the abstract class 443
as this is implementing the abstract class 459
as this is implementing the abstract class 474
as this is implementing the abstract class 490
as this is implementing the abstract class 505
as this is implementing the abstract class 521
as this is implementing the abstract class 536
as this is implementing the abstract class 552
org.apache.commons.lang3.exception.ExceptionUtils Line
--No comment-- 156
--No comment-- 163
org.apache.commons.lang3.math.NumberUtils Line
--No comment-- 746
--No comment-- 763
--No comment-- 774
--No comment-- 779
--No comment-- 799
--No comment-- 804
--No comment-- 827
org.apache.commons.lang3.reflect.ConstructorUtils Line
- Swallow 253
org.apache.commons.lang3.reflect.FieldUtils Line
--No comment-- 118
--No comment-- 132
--No comment-- 184
org.apache.commons.lang3.reflect.MemberUtils Line
--No comment-- 64
org.apache.commons.lang3.reflect.MethodUtils Line
--No comment-- 639
- Swallow the exception 684
org.apache.commons.lang3.reflect.TypeUtilsTest Line
cast is required by Sun Java, but not by Eclipse 604
org.apache.commons.lang3.text.ExtendedMessageFormat Line
--No comment-- 346
org.apache.commons.lang3.text.ExtendedMessageFormatTest Line
FAILING - currently sub-formats not supported 152
FAILING - currently sub-formats not supported 169
org.apache.commons.lang3.time.StopWatch Line
As from v2.1, the methods protect against inappropriate calls. Thus you cannot now call stop before start, resume before suspend or unsplit before split. </p> 47