Cleaned up commit.

Author: ctjlewis

src/com/google/javascript/jscomp/parsing/parser/Scanner.java

@@ -917,49 +917,10 @@ private static boolean isCombiningMark(char ch) {
       // FE20–FE2F
       (0xFE20 <= ch & ch <= 0xFE2F)
     );
-    // TODO (ctjl): Implement in a more reliable and future-proofed way, i.e.:
-    // return Character.getType(ch) == Character.NON_SPACING_MARK;
-  }
-
-  // TODO (ctjl): Implement
-  private static boolean isConnectorPunctuation() {
-    return true;
-  }
-
-  // TODO (ctjl): Implement
-  private static boolean isZeroWidthJoiner() {
-    return true;
-  }
-
-  // TODO (ctjl): Implement
-  private static boolean isZeroWidthNonJoiner() {
-    return true;
   }
 
   @SuppressWarnings("ShortCircuitBoolean") // Intentional to minimize branches in this code
   private static boolean isIdentifierPart(char ch) {
-    /**
-      https://www.ecma-international.org/ecma-262/5.1/#sec-7.6
-      IdentifierPart ::
-        IdentifierStart
-        ✓ isIdentifierPart()
-
-        UnicodeCombiningMark
-        ✓ isCombiningMark()
-
-        UnicodeDigit
-        ✓ Character.isDigit()
-
-        UnicodeConnectorPunctuation
-        ✓ isConnectorPunctuation()
-
-        <ZWNJ>
-        ✓ isZeroWidthNonJoiner()
-          
-        <ZWJ>
-        ✓ isZeroWidthJoiner()
-     */
-
     // Most code is written in pure ASCII, so create a fast path here.
     if (ch <= 127) {
       return ((ch >= 'A' & ch <= 'Z')
@@ -970,7 +931,7 @@ private static boolean isIdentifierPart(char ch) {
 
     // Handle non-ASCII characters.
     // TODO(tjgq): This should include all characters with the ID_Continue property, plus
-    // TODO(ctjl): Implement remaining grammar (zero-width joiners, etc.)
+    // Zero Width Non-Joiner and Zero Width Joiner.
     return isIdentifierStart(ch)
         || isCombiningMark(ch)
         || Character.isDigit(ch);