From 3de02d541d4e8f268d795b18e4edd2e37d86f7f4 Mon Sep 17 00:00:00 2001 From: preciz Date: Thu, 13 Aug 2026 17:09:08 +0200 Subject: [PATCH] Optimize URI.append_query field access Bind the current query in the function head so generated code can reuse the field instead of emitting repeated dynamic access paths. Assisted-by: Codex:GPT-5 --- lib/elixir/lib/uri.ex | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/elixir/lib/uri.ex b/lib/elixir/lib/uri.ex index bf27076ac84..3a32da98ffa 100644 --- a/lib/elixir/lib/uri.ex +++ b/lib/elixir/lib/uri.ex @@ -1059,11 +1059,11 @@ defmodule URI do %{uri | query: query} end - def append_query(%URI{} = uri, query) when is_binary(query) do - if String.ends_with?(uri.query, "&") do - %{uri | query: uri.query <> query} + def append_query(%URI{query: current} = uri, query) when is_binary(query) do + if String.ends_with?(current, "&") do + %{uri | query: current <> query} else - %{uri | query: uri.query <> "&" <> query} + %{uri | query: current <> "&" <> query} end end