Remove instances of broadcasting

While broadcasting is convenient syntax, it's tough on the compiler, and
so likely not worth it in library code like this.

Author: tecosaur

src/faces.jl

@@ -170,9 +170,12 @@ function Face(; font::Union{Nothing, String} = nothing,
          else inherit end)
 end
 
-Base.:(==)(a::Face, b::Face) =
-    getfield.(Ref(a), fieldnames(Face)) ==
-    getfield.(Ref(b), fieldnames(Face))
+function Base.:(==)(a::Face, b::Face)
+    for f in fieldnames(Face)
+        getfield(a, f) == getfield(b, f) || return false
+    end
+    true
+end
 
 Base.hash(f::Face, h::UInt) =
     mapfoldr(Base.Fix1(getfield, f), hash, fieldnames(Face), init=hash(Face, h))
@@ -714,7 +717,7 @@ function Base.convert(::Type{Face}, spec::Dict{String,Any})
          elseif spec["inherit"] isa String
              [Symbol(spec["inherit"]::String)]
          elseif spec["inherit"] isa Vector{String}
-             Symbol.(spec["inherit"]::Vector{String})
+             [Symbol(name) for name in spec["inherit"]::Vector{String}]
          else
              Symbol[]
          end)

src/legacy.jl

@@ -93,7 +93,7 @@ legacy_color(color::Symbol) =
     end
 
 function legacy_color(color::String)
-    namedcolours = String.(NAMED_COLORS)
+    namedcolours = map(String, NAMED_COLORS)
     if color in namedcolours
         legacy_color(Symbol(color))
     elseif 0 <= (color256 = something(tryparse(Int, color), -1)) <= 255