From e65ace10e3c40d6fef4e9997311d88c900e84ced Mon Sep 17 00:00:00 2001
From: Daniel Gredler <dgredler@openjdk.org>
Date: Fri, 26 Dec 2025 11:58:48 +0000
Subject: [PATCH] 6517125: FontStrike.getGlyphVectorOutline() not used

Reviewed-by: prr, serb
---
 .../macosx/classes/sun/font/CStrike.java      |  6 ---
 .../macosx/classes/sun/font/NativeStrike.java |  8 +---
 .../classes/sun/font/CompositeStrike.java     | 41 +------------------
 .../classes/sun/font/FileFontStrike.java      |  6 ---
 .../share/classes/sun/font/FontStrike.java    | 18 +-------
 .../unix/classes/sun/font/DelegateStrike.java |  6 ---
 .../unix/classes/sun/font/NativeStrike.java   |  6 ---
 .../classes/sun/font/NativeStrike.java        |  8 +---
 8 files changed, 5 insertions(+), 94 deletions(-)

diff --git a/src/java.desktop/macosx/classes/sun/font/CStrike.java b/src/java.desktop/macosx/classes/sun/font/CStrike.java
index 4b0ac9fd8d3..bffb76fb1ad 100644
--- a/src/java.desktop/macosx/classes/sun/font/CStrike.java
+++ b/src/java.desktop/macosx/classes/sun/font/CStrike.java
@@ -222,12 +222,6 @@ public final class CStrike extends PhysicalStrike {
         return getNativeGlyphOutline(getNativeStrikePtr(), glyphCode, x, y);
     }
 
-    // should implement, however not called though any path that is publicly exposed
-    @Override
-    GeneralPath getGlyphVectorOutline(int[] glyphs, float x, float y) {
-        throw new Error("not implemented yet");
-    }
-
     // called from the Sun2D renderer
     @Override
     long getGlyphImagePtr(int glyphCode) {
diff --git a/src/java.desktop/macosx/classes/sun/font/NativeStrike.java b/src/java.desktop/macosx/classes/sun/font/NativeStrike.java
index 9a1b1ecec3d..4ea267a4bd7 100644
--- a/src/java.desktop/macosx/classes/sun/font/NativeStrike.java
+++ b/src/java.desktop/macosx/classes/sun/font/NativeStrike.java
@@ -47,7 +47,6 @@ public class NativeStrike extends PhysicalStrike {
         throw new RuntimeException("NativeFont not used on Windows");
     }
 
-
     @Override
     void getGlyphImagePtrs(int[] glyphCodes, long[] images,int  len) {
     }
@@ -81,14 +80,9 @@ public class NativeStrike extends PhysicalStrike {
     Rectangle2D.Float getGlyphOutlineBounds(int glyphCode) {
         return null;
     }
+
     @Override
     GeneralPath getGlyphOutline(int glyphCode, float x, float y) {
         return null;
     }
-
-    @Override
-    GeneralPath getGlyphVectorOutline(int[] glyphs, float x, float y) {
-        return null;
-    }
-
 }
diff --git a/src/java.desktop/share/classes/sun/font/CompositeStrike.java b/src/java.desktop/share/classes/sun/font/CompositeStrike.java
index be6092f61f8..9974332a314 100644
--- a/src/java.desktop/share/classes/sun/font/CompositeStrike.java
+++ b/src/java.desktop/share/classes/sun/font/CompositeStrike.java
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2003, 2005, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2003, 2025, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -175,43 +175,4 @@ public final class CompositeStrike extends FontStrike {
             return path;
         }
     }
-
-    /* The physical font slot for each glyph is encoded in the glyph ID
-     * To be as efficient as possible we find a run of glyphs from the
-     * same slot and create a temporary array of these glyphs decoded
-     * to the slot. The slot font is then queried for the GeneralPath
-     * for that run of glyphs. GeneralPaths from each run are appended
-     * to create the shape for the whole glyph array.
-     */
-    GeneralPath getGlyphVectorOutline(int[] glyphs, float x, float y) {
-        GeneralPath path = null;
-        GeneralPath gp;
-        int glyphIndex = 0;
-        int[] tmpGlyphs;
-
-        while (glyphIndex < glyphs.length) {
-            int start = glyphIndex;
-            int slot = glyphs[glyphIndex] >>> 24;
-            while (glyphIndex < glyphs.length &&
-                   (glyphs[glyphIndex+1] >>> 24) == slot) {
-                glyphIndex++;
-            }
-            int tmpLen = glyphIndex-start+1;
-            tmpGlyphs = new int[tmpLen];
-            for (int i=0;i<tmpLen;i++) {
-                tmpGlyphs[i] = glyphs[i] & SLOTMASK;
-            }
-            gp = getStrikeForSlot(slot).getGlyphVectorOutline(tmpGlyphs, x, y);
-            if (path == null) {
-                path = gp;
-            } else if (gp != null) {
-                path.append(gp, false);
-            }
-        }
-        if (path == null) {
-            return new GeneralPath();
-        } else {
-            return path;
-        }
-    }
 }
diff --git a/src/java.desktop/share/classes/sun/font/FileFontStrike.java b/src/java.desktop/share/classes/sun/font/FileFontStrike.java
index 01e682e8390..a195e0347c0 100644
--- a/src/java.desktop/share/classes/sun/font/FileFontStrike.java
+++ b/src/java.desktop/share/classes/sun/font/FileFontStrike.java
@@ -39,7 +39,6 @@ import java.util.concurrent.ConcurrentHashMap;
 import static sun.awt.SunHints.*;
 import sun.java2d.pipe.OutlineTextRenderer;
 
-
 public class FileFontStrike extends PhysicalStrike {
 
     /* fffe and ffff are values we specially interpret as meaning
@@ -931,11 +930,6 @@ public class FileFontStrike extends PhysicalStrike {
         return gp;
     }
 
-    GeneralPath getGlyphVectorOutline(int[] glyphs, float x, float y) {
-        return fileFont.getGlyphVectorOutline(pScalerContext,
-                                              glyphs, glyphs.length, x, y);
-    }
-
     protected void adjustPoint(Point2D.Float pt) {
         if (invertDevTx != null) {
             invertDevTx.deltaTransform(pt, pt);
diff --git a/src/java.desktop/share/classes/sun/font/FontStrike.java b/src/java.desktop/share/classes/sun/font/FontStrike.java
index 2eff2f8169f..a02ba58b995 100644
--- a/src/java.desktop/share/classes/sun/font/FontStrike.java
+++ b/src/java.desktop/share/classes/sun/font/FontStrike.java
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2003, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2003, 2025, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -32,20 +32,12 @@ import java.awt.geom.Point2D;
 
 public abstract class FontStrike {
 
-
     protected FontStrikeDisposer disposer;
     protected FontStrikeDesc desc;
     protected StrikeMetrics strikeMetrics;
     protected boolean algoStyle = false;
     protected float boldness = 1f;
     protected float italic = 0f;
-    /*
-     * lastLookupTime is updated by Font2D.getStrike and can be used to
-     * choose strikes that have not been newly referenced for purging when
-     * memory usage gets too high. Active strikes will never be purged
-     * because purging is via GC of WeakReferences.
-     */
-    //protected long lastlookupTime/* = System.currentTimeMillis()*/;
 
     public abstract int getNumGlyphs();
 
@@ -70,11 +62,5 @@ public abstract class FontStrike {
 
     abstract Rectangle2D.Float getGlyphOutlineBounds(int glyphCode);
 
-    abstract GeneralPath
-        getGlyphOutline(int glyphCode, float x, float y);
-
-    abstract GeneralPath
-        getGlyphVectorOutline(int[] glyphs, float x, float y);
-
-
+    abstract GeneralPath getGlyphOutline(int glyphCode, float x, float y);
 }
diff --git a/src/java.desktop/unix/classes/sun/font/DelegateStrike.java b/src/java.desktop/unix/classes/sun/font/DelegateStrike.java
index 4535c60367d..201ffebbb0d 100644
--- a/src/java.desktop/unix/classes/sun/font/DelegateStrike.java
+++ b/src/java.desktop/unix/classes/sun/font/DelegateStrike.java
@@ -116,10 +116,4 @@ final class DelegateStrike extends NativeStrike {
     GeneralPath getGlyphOutline(int glyphCode, float x, float y) {
         return delegateStrike.getGlyphOutline(glyphCode, x, y);
     }
-
-    @Override
-    GeneralPath getGlyphVectorOutline(int[] glyphs, float x, float y) {
-        return delegateStrike.getGlyphVectorOutline(glyphs, x, y);
-    }
-
 }
diff --git a/src/java.desktop/unix/classes/sun/font/NativeStrike.java b/src/java.desktop/unix/classes/sun/font/NativeStrike.java
index 1f0937d504e..ef2e70f61ee 100644
--- a/src/java.desktop/unix/classes/sun/font/NativeStrike.java
+++ b/src/java.desktop/unix/classes/sun/font/NativeStrike.java
@@ -295,10 +295,4 @@ class NativeStrike extends PhysicalStrike {
      GeneralPath getGlyphOutline(int glyphCode, float x, float y) {
          return new GeneralPath();
      }
-
-     @Override
-     GeneralPath getGlyphVectorOutline(int[] glyphs, float x, float y) {
-         return new GeneralPath();
-     }
-
 }
diff --git a/src/java.desktop/windows/classes/sun/font/NativeStrike.java b/src/java.desktop/windows/classes/sun/font/NativeStrike.java
index 6e804020b55..eaf2eb5532f 100644
--- a/src/java.desktop/windows/classes/sun/font/NativeStrike.java
+++ b/src/java.desktop/windows/classes/sun/font/NativeStrike.java
@@ -47,7 +47,6 @@ public class NativeStrike extends PhysicalStrike {
         throw new RuntimeException("NativeFont not used on Windows");
     }
 
-
     @Override
     void getGlyphImagePtrs(int[] glyphCodes, long[] images,int  len) {
     }
@@ -81,14 +80,9 @@ public class NativeStrike extends PhysicalStrike {
     Rectangle2D.Float getGlyphOutlineBounds(int glyphCode) {
         return null;
     }
+
     @Override
     GeneralPath getGlyphOutline(int glyphCode, float x, float y) {
         return null;
     }
-
-    @Override
-    GeneralPath getGlyphVectorOutline(int[] glyphs, float x, float y) {
-        return null;
-    }
-
 }