Refactor away the few instances of broadcasting

They're harder on the compiler than list comprehensions or mapping.
tecosaur · Jul 31, 2024 · 3f9394f · 3f9394f
1 parent e1458b7
commit 3f9394f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/src/internals.jl b/src/internals.jl
@@ -104,7 +104,7 @@ function resolvedirpath(basedir::String, pathcomponents::Union{Tuple, AbstractVe
 end
 
 function resolvedirpaths(basedirs::Vector{String}, pathcomponents::Union{Tuple, AbstractVector}; create::Bool=false, existent::Bool=false)
- allpaths = resolvedirpath.(basedirs, Ref(pathcomponents); create)
+ allpaths = [resolvedirpath(bdir, pathcomponents; create) for bdir in basedirs]
  if existent
  filter(ispath, allpaths)
  else

diff --git a/src/unix.jl b/src/unix.jl
@@ -58,12 +58,11 @@ function reload()
  @setxdg PUBLICSHARE_DIR get(userdirs, :XDG_PUBLICSHARE_DIR, "~/Public")
  # Other directories
  FONTS_DIRS[] =
- append!([joinpath(DATA_HOME[], "fonts"),
- expanduser("~/.fonts")],
- joinpath.(DATA_DIRS[], "fonts")) |> unique!
+ append!([joinpath(DATA_HOME[], "fonts"), expanduser("~/.fonts")],
+ [joinpath(d, "fonts") for d in DATA_DIRS[]]) |> unique!
  APPLICATIONS_DIRS[] =
  append!([joinpath(DATA_HOME[], "applications")],
- joinpath.(DATA_DIRS[], "applications")) |> unique!
+ [joinpath(d, "applications") for d in DATA_DIRS[]]) |> unique!
  nothing
 end