Fix hashing of Dates.Time. Fixes #29480

JuliaLang · Oct 4, 2018 · 1229ee4 · 1229ee4
1 parent 5bc3de4
commit 1229ee4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/stdlib/Dates/src/types.jl b/stdlib/Dates/src/types.jl
@@ -350,6 +350,7 @@ function ==(a::Time, b::Time)
         microsecond(a) == microsecond(b) && nanosecond(a) == nanosecond(b)
 end
 (==)(x::TimeType, y::TimeType) = (===)(promote(x, y)...)
+Base.hash(x::Time, h::UInt) = hash(hour(x), hash(minute(x), hash(second(x), h)))
 
 import Base: sleep, Timer, timedwait
 sleep(time::Period) = sleep(toms(time) / 1000)

diff --git a/stdlib/Dates/test/types.jl b/stdlib/Dates/test/types.jl
@@ -194,6 +194,9 @@ c = Dates.Time(0)
     @test isfinite(Dates.Date)
     @test isfinite(Dates.DateTime)
     @test isfinite(Dates.Time)
+    @test c == c
+    @test c == (c + Dates.Hour(24))
+    @test hash(c) == hash(c + Dates.Hour(24))
 end
 @testset "Date-DateTime conversion/promotion" begin
     global a, b, c, d