Minor fixes from code review

Author: rmaucher

java/jakarta/websocket/server/ServerEndpointConfig.java

@@ -113,7 +113,7 @@ private Builder(Class<?> endpointClass, String path) {
             this.path = path;
         }
 
- /**
+        /**
          * Builds and returns a {@link ServerEndpointConfig} with the configured settings.
          *
          * @return The configured server endpoint configuration

java/org/apache/catalina/core/StandardServer.java

@@ -827,7 +827,7 @@ public synchronized void storeContext(Context context) throws InstanceNotFoundEx
             ObjectName sname = new ObjectName("Catalina:type=StoreConfig");
             MBeanServer server = Registry.getRegistry(null).getMBeanServer();
             if (server.isRegistered(sname)) {
-                server.invoke(sname, "store", new Object[] { context }, new String[] { "java.lang.String" });
+                server.invoke(sname, "store", new Object[] { context }, new String[] { "org.apache.catalina.Context" });
             } else {
                 log.error(sm.getString("standardServer.storeConfig.notAvailable", sname));
             }

java/org/apache/catalina/tribes/group/AbsoluteOrder.java

@@ -171,7 +171,7 @@ protected int compareBytes(byte[] d1, byte[] d2) {
          * @return comparison result
          */
         protected int compareBytes(byte b1, byte b2) {
-            return compareInts(b1, b2);
+            return compareInts(b1 & 0xFF, b2 & 0xFF);
         }
 
         /**

java/org/apache/el/parser/AstIdentifier.java

@@ -88,7 +88,7 @@ public Object getValue(EvaluationContext ctx) throws ELException {
         } finally {
             // Always reset the flag to false so the optimisation is not applied
             // inappropriately
-            ctx.putContext(this.getClass(), Boolean.FALSE);
+            ctx.putContext(ELResolver.StandaloneIdentifierMarker.class, Boolean.FALSE);
         }
 
         if (ctx.isPropertyResolved()) {

java/org/apache/tomcat/websocket/WsFrameBase.java

@@ -357,7 +357,8 @@ private boolean processDataControl() throws IOException {
                         new CloseReason(CloseCodes.PROTOCOL_ERROR, sm.getString("wsFrame.oneByteCloseCode")));
             }
             if (controlBufferBinary.remaining() > 1) {
-                code = controlBufferBinary.getShort();
+                short rawCode = controlBufferBinary.getShort();
+                code = rawCode & 0xFFFF;
                 if (controlBufferBinary.remaining() > 0) {
                     CoderResult cr = utf8DecoderControl.decode(controlBufferBinary, controlBufferText, true);
                     if (cr.isError()) {