graham/prop
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(buildings): use explicit step in descending range in encode_quadkey

Browse source 
zoom..1 without a step raises a runtime warning in Elixir 1.19 and was
causing the BulkFetch Task to crash immediately. Changed to zoom..1//-1.
This commit is contained in:
Graham McIntire 2026-04-26 10:54:38 -05:00
parent b17ff838ca
commit a290c99184
No known key found for this signature in database
GPG key ID: F4ABF488E6029E59
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/microwaveprop/buildings/ms_footprints.ex
View file

@ -58,7 +58,7 @@ defmodule Microwaveprop.Buildings.MsFootprints do
end end
defp encode_quadkey(tx, ty, zoom) do defp encode_quadkey(tx, ty, zoom) do
Enum.reduce(zoom..1, "", fn i, acc -> Enum.reduce(zoom..1//-1, "", fn i, acc ->
mask = Bitwise.bsl(1, i - 1) mask = Bitwise.bsl(1, i - 1)
digit = digit_for(tx, ty, mask) digit = digit_for(tx, ty, mask)
acc <> Integer.to_string(digit) acc <> Integer.to_string(digit)