Skip to content

Commit

Permalink
[fontc] Simplify timing color code
Browse files Browse the repository at this point in the history
I was in here to add a new IR type and thought this could be cleaned up.

In addition to the cleanup, a tiny additional benefit of this change is
that gray is now the default, and you don't need to add an explicit case
anytime a new workid is added.
  • Loading branch information
cmyr committed Apr 10, 2024
1 parent 0ff8116 commit c9f5c1b
Showing 1 changed file with 1 addition and 35 deletions.
36 changes: 1 addition & 35 deletions fontc/src/timing.rs
Original file line number Diff line number Diff line change
Expand Up @@ -167,46 +167,12 @@ fn short_name(id: &AnyWorkId) -> &'static str {

fn color(id: &AnyWorkId) -> &'static str {
match id {
AnyWorkId::Fe(FeWorkIdentifier::Anchor(..)) => "gray",
AnyWorkId::Fe(FeWorkIdentifier::Features) => "#e18707",
AnyWorkId::Fe(FeWorkIdentifier::GlobalMetrics) => "gray",
AnyWorkId::Fe(FeWorkIdentifier::Glyph(..)) => "#830356",
AnyWorkId::Fe(FeWorkIdentifier::GlyphIrDelete(..)) => "gray",
AnyWorkId::Fe(FeWorkIdentifier::GlyphOrder) => "gray",
AnyWorkId::Fe(FeWorkIdentifier::KerningGroups) => "gray",
AnyWorkId::Fe(FeWorkIdentifier::KernInstance(..)) => "gray",
AnyWorkId::Fe(FeWorkIdentifier::PreliminaryGlyphOrder) => "gray",
AnyWorkId::Fe(FeWorkIdentifier::StaticMetadata) => "gray",
AnyWorkId::Be(BeWorkIdentifier::Avar) => "gray",
AnyWorkId::Be(BeWorkIdentifier::Cmap) => "gray",
AnyWorkId::Be(BeWorkIdentifier::Features) => "gray",
AnyWorkId::Be(BeWorkIdentifier::FeaturesAst) => "gray",
AnyWorkId::Be(BeWorkIdentifier::Font) => "gray",
AnyWorkId::Be(BeWorkIdentifier::Fvar) => "gray",
AnyWorkId::Be(BeWorkIdentifier::Gdef) => "gray",
AnyWorkId::Be(BeWorkIdentifier::Glyf) => "gray",
AnyWorkId::Be(BeWorkIdentifier::GlyfFragment(..)) => "#00c1c9",
AnyWorkId::Be(BeWorkIdentifier::Gpos) => "gray",
AnyWorkId::Be(BeWorkIdentifier::Gsub) => "gray",
AnyWorkId::Be(BeWorkIdentifier::Gvar) => "gray",
AnyWorkId::Be(BeWorkIdentifier::GvarFragment(..)) => "#008786",
AnyWorkId::Be(BeWorkIdentifier::Head) => "gray",
AnyWorkId::Be(BeWorkIdentifier::Hhea) => "gray",
AnyWorkId::Be(BeWorkIdentifier::Hmtx) => "v",
AnyWorkId::Be(BeWorkIdentifier::Hvar) => "gray",
AnyWorkId::Be(BeWorkIdentifier::GatherIrKerning) => "gray",
AnyWorkId::Be(BeWorkIdentifier::KernFragment(..)) => "gray",
AnyWorkId::Be(BeWorkIdentifier::GatherBeKerning) => "gray",
AnyWorkId::Be(BeWorkIdentifier::Marks) => "gray",
AnyWorkId::Be(BeWorkIdentifier::Loca) => "gray",
AnyWorkId::Be(BeWorkIdentifier::LocaFormat) => "gray",
AnyWorkId::Be(BeWorkIdentifier::Maxp) => "gray",
AnyWorkId::Be(BeWorkIdentifier::Mvar) => "gray",
AnyWorkId::Be(BeWorkIdentifier::Name) => "gray",
AnyWorkId::Be(BeWorkIdentifier::Os2) => "gray",
AnyWorkId::Be(BeWorkIdentifier::Post) => "gray",
AnyWorkId::Be(BeWorkIdentifier::Stat) => "gray",
AnyWorkId::InternalTiming(..) => "#009a00",
_ => "gray",
}
}

Expand Down

0 comments on commit c9f5c1b

Please sign in to comment.