Registry + parser fixes: shader() ptr overload, looksLikeTemplateArgList, fn-ptr disambiguation

Parser:
- looksLikeTemplateArgList: parenDepth going negative now returns false immediately,
  fixing val<lo) being misidentified as template args in if-condition expressions
- looksLikeFunctionPointerDeclarator: (*name) without following ( or [ is NOT a
  function pointer (fixes shader(*noiseShader) misparse)
- parseClassMember: trailing requires clause after template<> consumed before member

CodeGen:
- PostfixExpr ++/-- normalized to prefix form (Rule C)
- Float literals get f suffix (Rule D)

CppBuild:
- fixColorTypes: skip single-letter vars and loop counters from color rewrite (Rule F)
- E0006: skip single-letter and two-letter-lowercase identifiers
- E0007: warn on 'color' variable name collision
- translate(0,0) no-op removal (Problem 7.2)
- checkForProcessingNameCollisions added

Processing.h:
- shader(PShader*), texture(PImage*), addChild(PShape*) pointer overloads
- PShader::set() double overloads for 1-4 args

Author: pepc84

mode/CppMode.jar

@@ -40,6 +40,12 @@ public class CppBuild {
   // Processing.h defines these class/struct names in namespace Processing.
   // If the user also defines them, we skip the user's version to avoid
   // "redefinition" errors -- the engine's definition takes precedence.
+  // Processing API names that are dangerous as user variable names (Problem 5.2/6.1)
+  // Processing API names confirmed to cause crashes/errors when used as variable names
+  // Kept minimal to avoid false positives -- only add names with confirmed crash reports
+  private static final java.util.Set<String> PROCESSING_API_NAMES = java.util.Set.of(
+      "color"   // confirmed: Processing::color typedef causes operator++ crash
+  );
   private static final java.util.Set<String> RESERVED_TYPES = java.util.Set.of(
       "Array", "ArrayList", "IntList", "FloatList", "StringList",
       "PVector", "PImage", "PGraphics", "PShape", "PFont", "PShader",
@@ -1023,11 +1029,15 @@ private File writeSketchImpl(RunnerListener listener) throws IOException {
 
     String code = sanitize(raw.toString());
     code = removeUserIncludes(code);
+    // Remove no-op translate(0, 0) and translate(0, 0, 0) calls (Problem 7.2)
+    code = code.replaceAll("\\btranslate\\s*\\(\\s*0+\\.?0*f?\\s*,\\s*0+\\.?0*f?\\s*\\)\\s*;", "");
+    code = code.replaceAll("\\btranslate\\s*\\(\\s*0+\\.?0*f?\\s*,\\s*0+\\.?0*f?\\s*,\\s*0+\\.?0*f?\\s*\\)\\s*;", "");
     warnReservedNames(code, listener);
     checkForUnsupportedJavaArraySyntax(code, listener);
     checkForArrayListGetValueCopy(code, listener);
     checkForArrayListGetDotAccess(code, listener);
     checkForJavaStaticCallSyntax(code, listener);
+    checkForProcessingNameCollisions(code, listener);
     code = stripRawStringLiterals(code);
     code = code.replaceAll("(?<=[0-9a-fA-FxXbB])'(?=[0-9a-fA-F])", "");
     code = javaToC(code);
@@ -2459,6 +2469,32 @@ private void checkForJavaStaticCallSyntax(String code, RunnerListener listener)
     }
   }
 
+  // [E0007] Variable name collides with Processing API function name
+  private void checkForProcessingNameCollisions(String code, RunnerListener listener) {
+    // Match variable declarations: "type name" or "type name =" at statement start
+    java.util.regex.Pattern varDecl = java.util.regex.Pattern.compile(
+        "\\b(?:int|float|bool|double|char|auto|color|String)\\s+("
+        + String.join("|", PROCESSING_API_NAMES)
+        + ")\\b");
+    java.util.regex.Matcher m = varDecl.matcher(code);
+    while (m.find()) {
+      String name = m.group(1);
+      // Find line number
+      int line = 1;
+      for (int i = 0; i < m.start(); i++) if (code.charAt(i) == '\n') line++;
+      String url = getWebsiteBaseUrl() + "/error/E0007.html";
+      String msg =
+        "\n[E0007] Variable name '" + name + "' shadows the Processing type 'color'.\n"
+        + "  Line " + line + ": this causes type inference errors. Rename to e.g. 'col', 'clr', or 'c2'.\n"
+        + "  Reference: " + url + "\n";
+      System.err.println(msg);
+      listener.statusError("E0007: '" + name + "' shadows Processing API -- see console");
+      throw new AlreadyReportedException("E0007: variable name shadows Processing API");
+    }
+  }
+
+
+
   private void checkForUnsupportedJavaArraySyntax(String code, RunnerListener listener) {
     try {
       CppJavaArrayCheck.check(code);

src/java/CppBuild.java

@@ -870,6 +870,13 @@ private List<TopLevelItem> parseClassMember(List<CppLexerToken> leadingComments,
         if (checkKeyword("template")) {
             templateParams = parseTemplateParamList();
             consumeLeadingComments();
+            // Trailing requires clause on template member: "template<T>\n    requires Concept<T>"
+            if (check(CppLexerTokenType.IDENTIFIER) && peek().text().equals("requires")) {
+                advance(); // consume requires
+                if (checkPunct("(")) { advance(); int _d=1; while(!isAtEnd()&&_d>0){if(checkPunct("("))_d++;else if(checkPunct(")"))_d--;advance();} }
+                else { int _d=0; while(!isAtEnd()){if(checkOp("<"))_d++;else if(checkOp(">")&&_d>0)_d--;else if(_d==0&&(checkPunct("{;")||checkPunct(";")|| checkKeyword("auto")||checkKeyword("void")||checkKeyword("bool")||checkKeyword("int")||checkKeyword("float")||checkKeyword("const")||checkKeyword("inline")||checkKeyword("static")||checkKeyword("virtual")||checkKeyword("constexpr")||checkKeyword("explicit")||checkKeyword("operator")))break;advance();} }
+                consumeLeadingComments();
+            }
         }
         if (checkKeyword("class") || checkKeyword("struct")) {
             // Anonymous struct: "struct { float x, y; } position;"
@@ -1117,13 +1124,10 @@ private List<TopLevelItem> parseFunctionOrVariable(List<CppLexerToken> leadingCo
 
         boolean isVirtual = matchKeyword("virtual");
         boolean isStatic = matchKeyword("static");
-        // Java "final" → C++ constexpr (for literals) or const
-        boolean isFinal = check(CppLexerTokenType.IDENTIFIER) && peek().text().equals("final");
-        if (isFinal) advance();
         matchKeyword("inline");
         matchKeyword("volatile"); // consume volatile qualifier
-        boolean isConst = matchKeyword("const") || isFinal;
-        boolean isConstexprFn = matchKeyword("constexpr") || matchKeyword("consteval") || isFinal;
+        boolean isConst = matchKeyword("const");
+        boolean isConstexprFn = matchKeyword("constexpr") || matchKeyword("consteval");
         if (isConstexprFn && !isConst) isConst = true;
         matchKeyword("constinit");
         if (!isVirtual) isVirtual = matchKeyword("virtual"); // constexpr virtual
@@ -2048,8 +2052,17 @@ private boolean looksLikeFunctionPointerDeclarator() {
         if (tokens.get(pos + i).type() != CppLexerTokenType.IDENTIFIER) return false;
         i++; // past name
         if (pos + i >= tokens.size()) return false;
-        // Either ) directly, or [N] then )
-        if (tokens.get(pos + i).isPunct(")")) return true;
+        // Either ) directly followed by ( for params, or [N] then )
+        // A bare "(*name)" MUST be followed by "(" (param list) to be a function pointer.
+        // "shader(*noiseShader);" is a call expression, not a fn ptr -- no "(" after ")".
+        if (tokens.get(pos + i).isPunct(")")) {
+            int j = i + 1;
+            if (j >= tokens.size()) return false;
+            // Function pointer: "void (*fp)(int)" -- "(" must follow
+            // Array of fn ptrs: "void (*fp)[N]" -- "[" may follow
+            return tokens.get(pos + j).isPunct("(")
+                || tokens.get(pos + j).isPunct("[");
+        }
         if (tokens.get(pos + i).isPunct("[")) return true; // array of fn ptrs
         return false;
     }
@@ -2789,7 +2802,7 @@ private boolean looksLikeTemplateArgList() {
             int steps = 0;
             while (!isAtEnd() && depth > 0 && steps < 120) {
                 if (checkPunct("(")) parenDepth++;
-                else if (checkPunct(")")) parenDepth--;
+                else if (checkPunct(")")) { parenDepth--; if (parenDepth < 0) return false; }
                 else if (checkPunct("{")) braceDepth++;
                 else if (checkPunct("}")) braceDepth--;
                 else if (parenDepth == 0 && braceDepth == 0) {