Author: ssteiner
Date: Thu Jun 16 15:38:26 2022
New Revision: 1901980
URL: http://svn.apache.org/viewvc?rev=1901980&view=rev
Log:
FOP-3078: Fix position of macron glyph
Modified:
xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/complexscripts/scripts/DefaultScriptProcessor.java
xmlgraphics/fop/trunk/fop-core/src/test/java/org/apache/fop/complexscripts/scripts/DefaultScriptTestCase.java
Modified:
xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/complexscripts/scripts/DefaultScriptProcessor.java
URL:
http://svn.apache.org/viewvc/xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/complexscripts/scripts/DefaultScriptProcessor.java?rev=1901980&r1=1901979&r2=1901980&view=diff
==============================================================================
---
xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/complexscripts/scripts/DefaultScriptProcessor.java
(original)
+++
xmlgraphics/fop/trunk/fop-core/src/main/java/org/apache/fop/complexscripts/scripts/DefaultScriptProcessor.java
Thu Jun 16 15:38:26 2022
@@ -103,45 +103,29 @@ public class DefaultScriptProcessor exte
CharAssociation ba = null;
int bg = -1;
int[] bpa = null;
+ int[] order = new int[ng];
for (int i = 0; i < ng; i++) {
- int gid = ga [ i ];
- int[] pa = (gpa != null) ? gpa [ i ] : null;
- CharAssociation ca = aa [ i ];
+ order[i] = i;
+ }
+ for (int i = 1; i < ng; i++) {
+ int[] pa = (gpa != null) ? gpa[i] : null;
if (isReorderedMark(gdef, ga, unscaledWidths, i, pa)) {
- nga [ k ] = gid;
- naa [ k ] = ca;
- if (npa != null) {
- npa [ k ] = pa;
- }
- k++;
- } else {
- if (bg != -1) {
- nga [ k ] = bg;
- naa [ k ] = ba;
- if (npa != null) {
- npa [ k ] = bpa;
+ for (int j = i; j > 0; j--) {
+ switchGlyphs(order, j);
+ int[] paj = (gpa != null) ? gpa[order[j]] : null;
+ if (!isMarkNotReordered(gdef, ga, order[j], paj)) {
+ break;
}
- k++;
- bg = -1;
- ba = null;
- bpa = null;
- }
- if (bg == -1) {
- bg = gid;
- ba = ca;
- bpa = pa;
}
}
}
- if (bg != -1) {
- nga [ k ] = bg;
- naa [ k ] = ba;
- if (npa != null) {
- npa [ k ] = bpa;
+ for (int i = 0; i < ng; i++) {
+ nga[i] = ga[order[i]];
+ naa[i] = aa[order[i]];
+ if (gpa != null) {
+ npa[i] = gpa[order[i]];
}
- k++;
}
- assert k == ng;
if (npa != null) {
System.arraycopy(npa, 0, gpa, 0, ng);
}
@@ -151,9 +135,22 @@ public class DefaultScriptProcessor exte
}
}
+ private static void switchGlyphs(int[] order, int i) {
+ if (i < 1) {
+ throw new IllegalArgumentException(i + " must be > 0");
+ }
+ int oi = order[i];
+ order[i] = order[i - 1];
+ order[i - 1] = oi;
+ }
+
protected boolean isReorderedMark(GlyphDefinitionTable gdef, int[] glyphs,
int[] unscaledWidths, int index, int[] pa) {
pa = (pa != null) ? pa : new int[1];
return gdef.isGlyphClass(glyphs[index],
GlyphDefinitionTable.GLYPH_CLASS_MARK) && pa[0] != 0;
}
+ private boolean isMarkNotReordered(GlyphDefinitionTable gdef, int[]
glyphs, int index, int[] pa) {
+ pa = (pa != null) ? pa : new int[1];
+ return gdef.isGlyphClass(glyphs[index],
GlyphDefinitionTable.GLYPH_CLASS_MARK) && pa[0] == 0;
+ }
}
Modified:
xmlgraphics/fop/trunk/fop-core/src/test/java/org/apache/fop/complexscripts/scripts/DefaultScriptTestCase.java
URL:
http://svn.apache.org/viewvc/xmlgraphics/fop/trunk/fop-core/src/test/java/org/apache/fop/complexscripts/scripts/DefaultScriptTestCase.java?rev=1901980&r1=1901979&r2=1901980&view=diff
==============================================================================
---
xmlgraphics/fop/trunk/fop-core/src/test/java/org/apache/fop/complexscripts/scripts/DefaultScriptTestCase.java
(original)
+++
xmlgraphics/fop/trunk/fop-core/src/test/java/org/apache/fop/complexscripts/scripts/DefaultScriptTestCase.java
Thu Jun 16 15:38:26 2022
@@ -54,11 +54,14 @@ public class DefaultScriptTestCase {
@Test
public void testProcessorReorder2() {
- String in = "\u00F6\u0323";
- int[][] gpa = new int[2][2];
- gpa[1][1] = 1;
+ String in = "S\u0323\u0323;";
+ int[][] gpa = new int[4][1];
+ gpa[2][0] = 1;
String actual = getFont().reorderCombiningMarks(in, gpa,
OTFScript.DEFAULT, null, null).toString();
- Assert.assertEquals(actual.charAt(0), 57344);
+ Assert.assertEquals(actual.charAt(0), 803);
+ Assert.assertEquals(actual.charAt(1), 57344);
+ Assert.assertEquals(actual.charAt(2), 803);
+ Assert.assertEquals(actual.charAt(3), 57344);
}
private MultiByteFont getFont() {
---------------------------------------------------------------------
To unsubscribe, e-mail: fop-commits-unsubscr...@xmlgraphics.apache.org
For additional commands, e-mail: fop-commits-h...@xmlgraphics.apache.org
