andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Replace several magic numbers. (#10094)

Browse Source 
Motivation:

Magic numbers seem hard to read or understand.

Modification:

Replace several magic numbers with named constants.

Result:

Improve readability and make it easier to maintain.
This commit is contained in:
feijermu 2020-03-09 18:48:59 +08:00 committed by Norman Maurer
parent ab14e0b583
commit 217365dd65
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
common/src/main/java/io/netty/util/internal/InternalThreadLocalMap.java
View File

@ -44,6 +44,8 @@ public final class InternalThreadLocalMap extends UnpaddedInternalThreadLocalMap
private static final int DEFAULT_ARRAY_LIST_INITIAL_CAPACITY = 8;
private static final int STRING_BUILDER_INITIAL_SIZE;
private static final int STRING_BUILDER_MAX_SIZE;
private static final int HANDLER_SHARABLE_CACHE_INITIAL_CAPACITY = 4;
private static final int INDEXED_VARIABLE_TABLE_INITIAL_SIZE = 32;
public static final Object UNSET = new Object();
@ -128,7 +130,7 @@ public final class InternalThreadLocalMap extends UnpaddedInternalThreadLocalMap
}
private static Object[] newIndexedVariableTable() {
Object[] array = new Object[32];
Object[] array = new Object[INDEXED_VARIABLE_TABLE_INITIAL_SIZE];
Arrays.fill(array, UNSET);
return array;
}
@ -259,7 +261,7 @@ public final class InternalThreadLocalMap extends UnpaddedInternalThreadLocalMap
Map<Class<?>, Boolean> cache = handlerSharableCache;
if (cache == null) {
// Start with small capacity to keep memory overhead as low as possible.
handlerSharableCache = cache = new WeakHashMap<>(4);
handlerSharableCache = cache = new WeakHashMap<>(HANDLER_SHARABLE_CACHE_INITIAL_CAPACITY);
}
return cache;
}