(commons-imaging) 02/03: Use final

[ggregory]

This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-imaging.git

commit 154174857f2ed3abff9b7916242ff64d6e9de476
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Sat Dec 23 11:03:23 2023 -0500

    Use final
    
    Remove paren noise
---
 .../apache/commons/imaging/formats/bmp/AbstractPixelParser.java  | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git 
a/src/main/java/org/apache/commons/imaging/formats/bmp/AbstractPixelParser.java 
b/src/main/java/org/apache/commons/imaging/formats/bmp/AbstractPixelParser.java
index b5fee061..bdb81586 100644
--- 
a/src/main/java/org/apache/commons/imaging/formats/bmp/AbstractPixelParser.java
+++ 
b/src/main/java/org/apache/commons/imaging/formats/bmp/AbstractPixelParser.java
@@ -40,17 +40,14 @@ abstract class AbstractPixelParser {
         is = new ByteArrayInputStream(imageData);
     }
 
-    int getColorTableRgb(int actual) {
-        actual *= 4;
+    int getColorTableRgb(final int index) {
+        final int actual = index * 4;
         final int blue = 0xff & colorTable[actual + 0];
         final int green = 0xff & colorTable[actual + 1];
         final int red = 0xff & colorTable[actual + 2];
         final int alpha = 0xff;
 
-        return (alpha << 24)
-                | (red << 16)
-                | (green << 8)
-                | (blue << 0);
+        return alpha << 24 | red << 16 | green << 8 | blue << 0;
     }
 
     public abstract void processImage(ImageBuilder imageBuilder) throws 
ImagingException, IOException;