[ignore] code cleanup

Author: adamretter

src/org/exist/xquery/functions/map/MapExpr.java

@@ -15,21 +15,22 @@
  */
 public class MapExpr extends AbstractExpression {
 
-    protected List<Mapping> mappings = new ArrayList(13);
+    private final List<Mapping> mappings = new ArrayList<>(13);
 
-    public MapExpr(XQueryContext context) {
+    public MapExpr(final XQueryContext context) {
         super(context);
     }
 
-    public void map(PathExpr key, PathExpr value) {
+    public void map(final PathExpr key, final PathExpr value) {
         final Mapping mapping = new Mapping(key.simplify(), value.simplify());
         this.mappings.add(mapping);
     }
 
-    public void analyze(AnalyzeContextInfo contextInfo) throws XPathException {
-        if(getContext().getXQueryVersion() < 30){
+    @Override
+    public void analyze(final AnalyzeContextInfo contextInfo) throws XPathException {
+        if (getContext().getXQueryVersion() < 30) {
             throw new XPathException(this, ErrorCodes.EXXQDY0003,
-                "Map is not available before XQuery 3.0");
+                    "Map is not available before XQuery 3.0");
         }
         contextInfo.setParent(this);
         for (final Mapping mapping : this.mappings) {
@@ -38,35 +39,40 @@ public void analyze(AnalyzeContextInfo contextInfo) throws XPathException {
         }
     }
 
-    public Sequence eval(Sequence contextSequence, Item contextItem) throws XPathException {
-        if (contextItem != null)
-            {contextSequence = contextItem.toSequence();}
+    @Override
+    public Sequence eval(Sequence contextSequence, final Item contextItem) throws XPathException {
+        if (contextItem != null) {
+            contextSequence = contextItem.toSequence();
+        }
         final MapType map = new MapType(this.context);
         for (final Mapping mapping : this.mappings) {
             final Sequence key = mapping.key.eval(contextSequence);
-            if (key.getItemCount() != 1)
-                {throw new XPathException(MapErrorCode.EXMPDY001, "Expected single value for key, got " + key.getItemCount());}
+            if (key.getItemCount() != 1) {
+                throw new XPathException(MapErrorCode.EXMPDY001, "Expected single value for key, got " + key.getItemCount());
+            }
             final AtomicValue atomic = key.itemAt(0).atomize();
             final Sequence value = mapping.value.eval(contextSequence);
             map.add(atomic, value);
         }
         return map;
     }
 
+    @Override
     public int returnsType() {
         return Type.MAP;
     }
 
     @Override
-    public void accept(ExpressionVisitor visitor) {
+    public void accept(final ExpressionVisitor visitor) {
         super.accept(visitor);
-        for (final Mapping mapping: this.mappings) {
+        for (final Mapping mapping : this.mappings) {
             mapping.key.accept(visitor);
             mapping.value.accept(visitor);
         }
     }
 
-    public void dump(ExpressionDumper dumper) {
+    @Override
+    public void dump(final ExpressionDumper dumper) {
         dumper.display("map {");
         for (final Mapping mapping : this.mappings) {
             mapping.key.dump(dumper);
@@ -77,7 +83,7 @@ public void dump(ExpressionDumper dumper) {
     }
 
     @Override
-    public void resetState(boolean postOptimization) {
+    public void resetState(final boolean postOptimization) {
         super.resetState(postOptimization);
         mappings.forEach(m -> m.resetState(postOptimization));
     }
@@ -86,12 +92,12 @@ private static class Mapping {
         final Expression key;
         final Expression value;
 
-        public Mapping(Expression key, Expression value) {
+        public Mapping(final Expression key, final Expression value) {
             this.key = key;
             this.value = value;
         }
 
-        private void resetState(boolean postOptimization) {
+        private void resetState(final boolean postOptimization) {
             key.resetState(postOptimization);
             value.resetState(postOptimization);
         }