Remove suppressions for checkstyle MagicNumber

Author: slawekjaranowski

maven-resolver-api/src/main/java/org/eclipse/aether/DefaultRepositorySystemSession.java

@@ -382,7 +382,6 @@ public DefaultRepositorySystemSession setTransferListener(TransferListener trans
         return this;
     }
 
-    @SuppressWarnings("checkstyle:magicnumber")
     private <T> Map<String, T> copySafe(Map<?, ?> table, Class<T> valueType) {
         Map<String, T> map;
         if (table == null || table.isEmpty()) {

maven-resolver-api/src/main/java/org/eclipse/aether/repository/AuthenticationDigest.java

@@ -146,7 +146,6 @@ public void update(String... strings) {
      *
      * @param chars The characters to update the digest with, may be {@code null}.
      */
-    @SuppressWarnings("checkstyle:magicnumber")
     public void update(char... chars) {
         if (chars != null) {
             for (char c : chars) {
@@ -167,7 +166,6 @@ public void update(byte... bytes) {
         }
     }
 
-    @SuppressWarnings("checkstyle:magicnumber")
     private String digest() {
         byte[] bytes = digest.digest();
         StringBuilder buffer = new StringBuilder(bytes.length * 2);

maven-resolver-demos/maven-resolver-demo-snippets/src/main/java/org/apache/maven/resolver/examples/util/ConsoleTransferListener.java

@@ -154,7 +154,6 @@ public void transferCorrupted(TransferEvent event) {
         event.getException().printStackTrace(out);
     }
 
-    @SuppressWarnings("checkstyle:magicnumber")
     protected long toKB(long bytes) {
         return (bytes + 1023) / 1024;
     }

maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/DefaultTrackingFileManager.java

@@ -132,7 +132,6 @@ private Object getMutex(File file) {
         }
     }
 
-    @SuppressWarnings({"checkstyle:magicnumber"})
     private FileLock fileLock(FileChannel channel, long size, boolean shared) throws IOException {
         FileLock lock = null;
         for (int attempts = 8; attempts >= 0; attempts--) {

maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/DefaultUpdatePolicyAnalyzer.java

@@ -48,7 +48,6 @@ public String getEffectiveUpdatePolicy(RepositorySystemSession session, String p
         return ordinalOfUpdatePolicy(policy1) < ordinalOfUpdatePolicy(policy2) ? policy1 : policy2;
     }
 
-    @SuppressWarnings({"checkstyle:magicnumber"})
     private int ordinalOfUpdatePolicy(String policy) {
         if (RepositoryPolicy.UPDATE_POLICY_DAILY.equals(policy)) {
             return 1440;
@@ -102,7 +101,6 @@ public boolean isUpdatedRequired(RepositorySystemSession session, long lastModif
         return checkForUpdates;
     }
 
-    @SuppressWarnings({"checkstyle:magicnumber"})
     private int getMinutes(String policy) {
         int minutes;
         try {

maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/collect/df/NodeStack.java

@@ -30,10 +30,7 @@
  */
 final class NodeStack {
 
-    @SuppressWarnings({"checkstyle:magicnumber"})
-    // CHECKSTYLE_OFF: MagicNumber
     ArrayList<DependencyNode> nodes = new ArrayList<>(96);
-    // CHECKSTYLE_ON: MagicNumber
 
     public DependencyNode top() {
         if (nodes.isEmpty()) {

maven-resolver-transport-http/src/main/java/org/eclipse/aether/transport/http/GlobalState.java

@@ -145,7 +145,6 @@ public HttpClientConnectionManager getConnectionManager(ConnMgrConfig config) {
         return connectionManagers.computeIfAbsent(config, GlobalState::newConnectionManager);
     }
 
-    @SuppressWarnings("checkstyle:magicnumber")
     public static HttpClientConnectionManager newConnectionManager(ConnMgrConfig connMgrConfig) {
         RegistryBuilder<ConnectionSocketFactory> registryBuilder = RegistryBuilder.<ConnectionSocketFactory>create()
                 .register("http", PlainConnectionSocketFactory.getSocketFactory());

maven-resolver-transport-http/src/main/java/org/eclipse/aether/transport/http/HttpTransporter.java

@@ -537,7 +537,6 @@ private void prepare(HttpUriRequest request, SharingHttpContext context) {
         }
     }
 
-    @SuppressWarnings("checkstyle:magicnumber")
     private void mkdirs(URI uri, SharingHttpContext context) {
         List<URI> dirs = UriUtils.getDirectories(baseUri, uri);
         int index = 0;
@@ -614,7 +613,6 @@ private <T extends HttpUriRequest> T commonHeaders(T request) {
         return request;
     }
 
-    @SuppressWarnings("checkstyle:magicnumber")
     private <T extends HttpUriRequest> void resume(T request, GetTask task) {
         long resumeOffset = task.getResumeOffset();
         if (resumeOffset > 0L && task.getDataFile() != null) {
@@ -626,7 +624,6 @@ private <T extends HttpUriRequest> void resume(T request, GetTask task) {
         }
     }
 
-    @SuppressWarnings("checkstyle:magicnumber")
     private void handleStatus(CloseableHttpResponse response) throws HttpResponseException {
         int status = response.getStatusLine().getStatusCode();
         if (status >= 300) {

maven-resolver-transport-http/src/main/java/org/eclipse/aether/transport/http/SharingAuthCache.java

@@ -40,7 +40,6 @@ final class SharingAuthCache implements AuthCache {
         authSchemes = new HashMap<>();
     }
 
-    @SuppressWarnings("checkstyle:magicnumber")
     private static HttpHost toKey(HttpHost host) {
         if (host.getPort() <= 0) {
             int port = host.getSchemeName().equalsIgnoreCase("https") ? 443 : 80;

maven-resolver-util/src/main/java/org/eclipse/aether/util/ChecksumUtils.java

@@ -145,7 +145,6 @@ private static Map<String, Object> calc(InputStream data, Collection<String> alg
      * @param bytes The bytes to represent in hex notation, may be be {@code null}.
      * @return The hexadecimal representation of the input or {@code null} if the input was {@code null}.
      */
-    @SuppressWarnings("checkstyle:magicnumber")
     public static String toHexString(byte[] bytes) {
         if (bytes == null) {
             return null;
@@ -172,7 +171,6 @@ public static String toHexString(byte[] bytes) {
      * @return The byte array of the input or {@code null} if the input was {@code null}.
      * @since 1.8.0
      */
-    @SuppressWarnings("checkstyle:magicnumber")
     public static byte[] fromHexString(String hexString) {
         if (hexString == null) {
             return null;

maven-resolver-util/src/main/java/org/eclipse/aether/util/graph/visitor/Stack.java

@@ -29,9 +29,7 @@
 class Stack<E> extends AbstractList<E> implements RandomAccess {
 
     @SuppressWarnings("unchecked")
-    // CHECKSTYLE_OFF: MagicNumber
     private E[] elements = (E[]) new Object[96];
-    // CHECKSTYLE_ON: MagicNumber
 
     private int size;
 

maven-resolver-util/src/main/java/org/eclipse/aether/util/repository/SecretAuthentication.java

@@ -71,7 +71,6 @@ private static char[] copy(char[] chars) {
         return (chars != null) ? chars.clone() : null;
     }
 
-    @SuppressWarnings("checkstyle:magicnumber")
     private char[] xor(char[] chars) {
         if (chars != null) {
             int mask = System.identityHashCode(this);

maven-resolver-util/src/main/java/org/eclipse/aether/util/version/UnionVersionRange.java

@@ -137,7 +137,6 @@ public boolean equals(Object obj) {
         return ranges.equals(that.ranges);
     }
 
-    @SuppressWarnings("checkstyle:magicnumber")
     @Override
     public int hashCode() {
         return 97 * ranges.hashCode();