Skip to content

Commit 156de8b

Browse files
committed
Fixed some comments in code
1 parent 660c05b commit 156de8b

2 files changed

Lines changed: 3 additions & 31 deletions

File tree

src/EPPlus.Fonts.OpenType.Tests/Validation/GsubTableValidationTests.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,9 @@ public class GsubTableValidationTests : FontTestBase
1010
public override TestContext? TestContext { get; set; }
1111

1212
[TestMethod]
13-
//[DataRow("Roboto")]
14-
//[DataRow("OpenSans")]
15-
//[DataRow("SourceSans3")]
13+
[DataRow("Roboto")]
14+
[DataRow("OpenSans")]
15+
[DataRow("SourceSans3")]
1616
[DataRow("NotoEmoji")]
1717
public void GsubTableValidation_Test(string fontName)
1818
{

src/EPPlus.Fonts.OpenType/TextShaping/Positioning/MarkToBaseProvider.cs

Lines changed: 0 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -36,19 +36,6 @@ public MarkToBaseProvider(OpenTypeFont font)
3636
{
3737
_subtables = new List<MarkToBaseSubTableFormat1>();
3838
}
39-
System.Diagnostics.Debug.WriteLine($"[MarkToBase] Initialized with {_subtables.Count} subtables");
40-
if (_subtables.Count > 0)
41-
{
42-
System.Diagnostics.Debug.WriteLine($"[MarkToBase] Subtable details:");
43-
foreach (var subtable in _subtables)
44-
{
45-
System.Diagnostics.Debug.WriteLine($" MarkCoverage: {subtable.MarkCoverage?.GetType().Name}");
46-
System.Diagnostics.Debug.WriteLine($" BaseCoverage: {subtable.BaseCoverage?.GetType().Name}");
47-
System.Diagnostics.Debug.WriteLine($" MarkClassCount: {subtable.MarkClassCount}");
48-
System.Diagnostics.Debug.WriteLine($" MarkArray.MarkCount: {subtable.MarkArray?.MarkCount ?? 0}");
49-
System.Diagnostics.Debug.WriteLine($" BaseArray.BaseCount: {subtable.BaseArray?.BaseCount ?? 0}");
50-
}
51-
}
5239
}
5340

5441
/// <summary>
@@ -61,38 +48,23 @@ public void ApplyMarkPositioning(List<ShapedGlyph> glyphs)
6148
if (_subtables.Count == 0 || glyphs.Count < 2)
6249
return;
6350

64-
System.Diagnostics.Debug.WriteLine($"[MarkToBase] Processing {glyphs.Count} glyphs");
65-
System.Diagnostics.Debug.WriteLine($"[MarkToBase] Found {_subtables.Count} subtables");
66-
6751
// Process glyphs left-to-right
6852
for (int i = 1; i < glyphs.Count; i++)
6953
{
7054
var baseGlyph = glyphs[i - 1];
7155
var markGlyph = glyphs[i];
7256

73-
System.Diagnostics.Debug.WriteLine($"[MarkToBase] Checking pair: base={baseGlyph.GlyphId}, mark={markGlyph.GlyphId}");
74-
7557
bool positioned = false;
7658

7759
// Try each subtable until we find positioning
7860
foreach (var subtable in _subtables)
7961
{
8062
if (TryPositionMark(subtable, baseGlyph, markGlyph))
8163
{
82-
System.Diagnostics.Debug.WriteLine($" ✓ Positioned! Mark now: XAdv={markGlyph.XAdvance}, XOff={markGlyph.XOffset}, YOff={markGlyph.YOffset}");
83-
84-
// Double check the glyph in the list
85-
System.Diagnostics.Debug.WriteLine($" Verify list[{i}]: XAdv={glyphs[i].XAdvance}, YOff={glyphs[i].YOffset}");
86-
//System.Diagnostics.Debug.WriteLine($" ✓ Positioned mark {markGlyph.GlyphId} over base {baseGlyph.GlyphId}");
8764
positioned = true;
8865
break;
8966
}
9067
}
91-
92-
if (!positioned)
93-
{
94-
System.Diagnostics.Debug.WriteLine($" ✗ No positioning found for mark {markGlyph.GlyphId}");
95-
}
9668
}
9769
}
9870

0 commit comments

Comments
 (0)