Skip to content

Prevent infinite recursion of unit by adding a method unit(::Type{Any}) #807

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 14 commits into from
Sep 15, 2025
Merged

Prevent infinite recursion of unit by adding a method unit(::Type{Any}) #807

Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
330869f
Define unit(::Any)m unit(::Funciton) and add a type restriction to un…
SeSodesa Aug 29, 2025
51e81a5
test/dates: assume NoUnits instead of MethodErrors in the case of Dat…
SeSodesa Aug 30, 2025
4f2bf7f
Loosen the type restrictions of unit(::Type{Missing,T})
SeSodesa Sep 1, 2025
c230a6a
Fully qualify Dates.FixedPeriod in type restriction of method unit(x:…
SeSodesa Sep 1, 2025
fe1e225
Revert the Dates-related tests to throw MethodErrors instead of testi…
SeSodesa Sep 1, 2025
156ffe2
If a unit call bubbles up to unit(::Any), throw a DomainError
SeSodesa Sep 3, 2025
b43536b
Remove now-unnecessary comment about type restriction from unit(::Typ…
SeSodesa Sep 3, 2025
e071b61
Remove method unit(::Any), and move unit(::Type{Any}) for docstring p…
SeSodesa Sep 3, 2025
108bc4e
Change Dates-related tests to expect MethodErrors again, after the re…
SeSodesa Sep 3, 2025
16c7404
Shorten error message thrown by unit(Any)
SeSodesa Sep 5, 2025
6320332
Replace docstring of unit(::Type{Any}) with a simple comment
SeSodesa Sep 5, 2025
6af1980
Take Julia 1.6 compatibility into account with the unit(Any) error me…
SeSodesa Sep 10, 2025
7ea4b1f
Follow code style of project
SeSodesa Sep 15, 2025
48b9629
Further code style changes
SeSodesa Sep 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions src/utils.jl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -145,6 +145,11 @@ true
@inline unit(x::Type{Missing}) = missing
@inline unit(x::Missing) = missing

# Prevent infinite recursion, in case unit(Any) is called.
@inline unit(x::Type{Any}) = throw(ArgumentError(
"unit(Any) was called, which has no meaningful result. This may be due to calling unit(eltype(…)), since eltype has a generic fallback method that returns Any."
))

"""
absoluteunit(::Units)
absoluteunit(::Quantity)
Expand Down
Loading