[ignore] Small code formatting improvements

Author: adamretter

exist-core/src/main/java/org/exist/dom/memtree/DocumentImpl.java

@@ -584,7 +584,8 @@ public NodeImpl getNode(final int nodeNum) throws DOMException {
 
     private NodeImpl referenceFromNodeProxy(final int nodeNum, final NodeProxy nodeProxy) {
         final NodeImpl<?> node;
-        switch (nodeProxy.getNodeType()) {
+        final int nodeType = nodeProxy.getNodeType();
+        switch (nodeType) {
             case Node.ELEMENT_NODE:
                 node = new ElementReferenceImpl(getExpression(), this, nodeNum, nodeProxy);
                 break;
@@ -651,7 +652,7 @@ public Element getDocumentElement() {
 
     @Override
     public Node getFirstChild() {
-        if(size > 1) {
+        if (size > 1) {
             return getNode(1);
         }
         return null;
@@ -706,9 +707,10 @@ public int getFirstChildFor(final int nodeNumber) {
 
         final short level = treeLevel[nodeNumber];
         final int nextNode = nodeNumber + 1;
-        if((nextNode < size) && (treeLevel[nextNode] > level)) {
+        if ((nextNode < size) && (treeLevel[nextNode] > level)) {
             return nextNode;
         }
+
         return -1;
     }
 

exist-core/src/main/java/org/exist/dom/persistent/DocumentImpl.java

@@ -988,7 +988,7 @@ public boolean hasChildNodes() {
     @EnsureContainerLocked(mode=READ_LOCK)
     public NodeList getChildNodes() {
         final org.exist.dom.NodeListImpl list = new org.exist.dom.NodeListImpl();
-        try(final DBBroker broker = pool.getBroker()) {
+        try (final DBBroker broker = pool.getBroker()) {
             for(int i = 0; i < children; i++) {
                 final Node child = broker.objectWith(new NodeProxy(getExpression(), this, NodeId.DOCUMENT_NODE, childAddress[i]));
                 list.add(child);
@@ -1515,8 +1515,9 @@ public String getBaseURI() {
 
     @Override
     public String toString() {
+        @Nullable final Element documentElement = getDocumentElement();
         return getURI() + " - <" +
-            (getDocumentElement() != null ? getDocumentElement().getNodeName() : null) + ">";
+            (documentElement != null ? documentElement.getNodeName() : null) + ">";
     }
 
     @Override

exist-core/src/main/java/org/exist/dom/persistent/NodeProxy.java

@@ -319,7 +319,7 @@ public QName getQName() {
         return qname;
     }
 
-    public void setQName(QName qname) {
+    public void setQName(final QName qname) {
         this.qname = qname;
     }