Fix LazyString building with non-ASCII preceding $ (#49248)

JuliaLang · Apr 5, 2023 · bd1a664 · bd1a664
1 parent 8be6f0f
commit bd1a664
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/base/strings/lazy.jl b/base/strings/lazy.jl
@@ -67,7 +67,7 @@ macro lazy_str(text)
     parts = Any[]
     lastidx = idx = 1
     while (idx = findnext('$', text, idx)) !== nothing
-        lastidx < idx && push!(parts, text[lastidx:idx-1])
+        lastidx < idx && push!(parts, text[lastidx:prevind(text, idx)])
         idx += 1
         expr, idx = Meta.parseatom(text, idx; filename=string(__source__.file))
         push!(parts, esc(expr))

diff --git a/test/strings/basic.jl b/test/strings/basic.jl
@@ -1191,6 +1191,9 @@ end
         end
         return a
     end |> Core.Compiler.is_foldable
+    let i=49248
+        @test String(lazy"PR n°$i") == "PR n°49248"
+    end
 end
 
 @testset "String Effects" begin