From daead75e9c41a92ab002d257ba91fca4a3732390 Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Mon, 13 Sep 2021 19:45:14 +0200 Subject: [PATCH] LibGfx/TTF: Ignore unsupported composite glyph flags Replace some program-stopping TODO()'s with FIXME comments, since many fonts actually render mostly fine without support for all features. --- Userland/Libraries/LibGfx/TrueTypeFont/Glyf.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Userland/Libraries/LibGfx/TrueTypeFont/Glyf.cpp b/Userland/Libraries/LibGfx/TrueTypeFont/Glyf.cpp index 7cdd39eb84..c7d2884f5a 100644 --- a/Userland/Libraries/LibGfx/TrueTypeFont/Glyf.cpp +++ b/Userland/Libraries/LibGfx/TrueTypeFont/Glyf.cpp @@ -171,16 +171,16 @@ Optional Glyf::Glyph::ComponentIterator::n e = arg1; f = arg2; } else { - TODO(); + // FIXME: Implement this. There's no TODO() here since many fonts work just fine without this. } if (flags & (u16)CompositeGlyfFlags::UseMyMetrics) { // FIXME: Implement this. There's no TODO() here since many fonts work just fine without this. } if (flags & (u16)CompositeGlyfFlags::ScaledComponentOffset) { - TODO(); + // FIXME: Implement this. There's no TODO() here since many fonts work just fine without this. } if (flags & (u16)CompositeGlyfFlags::UnscaledComponentOffset) { - TODO(); + // FIXME: Implement this. There's no TODO() here since many fonts work just fine without this. } m_has_more = (flags & (u16)CompositeGlyfFlags::MoreComponents); return Item {