Further improve warning message for range with negative step (#14070)

woylie · josevalim · commit 048cc2ddb178 · 2024-12-13T09:52:42.000+01:00
diff --git a/lib/elixir/lib/range.ex b/lib/elixir/lib/range.ex
@@ -192,7 +192,7 @@ defmodule Range do
         IO.warn_once(
           {__MODULE__, :new},
           fn ->
-            "Range.new/2 has a default step of -1 when last < first, please call Range.new/3 explicitly passing the step of -1 instead"
+            "Range.new/2 and first..last default to a step of -1 when last < first. Use Range.new(first, last, -1) or first..last//-1, or pass 1 if that was your intention"
           end,
           3
         )

Original file line number	Diff line number	Diff line change
`@@ -192,7 +192,7 @@ defmodule Range do`
`192`	`192`	`IO.warn_once(`
`193`	`193`	`{__MODULE__, :new},`
`194`	`194`	`fn ->`
`195`		`- "Range.new/2 has a default step of -1 when last < first, please call Range.new/3 explicitly passing the step of -1 instead"`
	`195`	`+ "Range.new/2 and first..last default to a step of -1 when last < first. Use Range.new(first, last, -1) or first..last//-1, or pass 1 if that was your intention"`
`196`	`196`	`end,`
`197`	`197`	`3`
`198`	`198`	`)`