🐛 Reduce allocations when using Crayons

We need to convert a Crayon to string outside the function to avoid allocations.
JuliaSpace · May 24, 2024 · fe04213 · fe04213
1 parent 2d515ff
commit fe04213
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/SatelliteToolboxSgp4.jl b/src/SatelliteToolboxSgp4.jl
@@ -24,9 +24,9 @@ include("types.jl")
 const _JD_1900 = DateTime(1900, 1, 1, 12, 0, 0) |> datetime2julian
 
 # Escape sequences related to the crayons.
-const _D = Crayon(reset = true)
-const _B = crayon"bold"
-const _Y = crayon"yellow bold"
+const _D = string(Crayon(reset = true))
+const _B = string(crayon"bold")
+const _Y = string(crayon"yellow bold")
 
 ############################################################################################
 #                                         Includes                                         #

diff --git a/src/tle.jl b/src/tle.jl
@@ -380,9 +380,9 @@ function fit_sgp4_tle!(
 
     # Check if stdout supports colors.
     has_color = get(stdout, :color, false)::Bool
-    cd = has_color ? string(_D) : ""
-    cb = has_color ? string(_B) : ""
-    cy = has_color ? string(_Y) : ""
+    cd = has_color ? _D : ""
+    cb = has_color ? _B : ""
+    cy = has_color ? _Y : ""
 
     # Assemble the weight matrix.
     W = Diagonal(