Skip to content

Commit 7c6ef20

Browse files
fix ambiguity
1 parent 86185a5 commit 7c6ef20

File tree

2 files changed

+15
-11
lines changed

2 files changed

+15
-11
lines changed

src/datetime.jl

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -296,8 +296,8 @@ for CFDateTime in (
296296
return (_origin_period(dt1) - _origin_period(dt2)) + (dt1.instant - dt2.instant)
297297
end
298298

299-
# fast case if the same resolution and time origin is used
300-
@eval @inline function -(dt1::$CFDateTime{T, Torigintuple}, dt2::$CFDateTime{T, Torigintuple}) where {T, Torigintuple}
299+
# fast case if the same time origin is used
300+
@eval @inline function -(dt1::$CFDateTime{T1, Torigintuple}, dt2::$CFDateTime{T2, Torigintuple}) where {T1, T2, Torigintuple}
301301
return dt1.instant - dt2.instant
302302
end
303303
end
@@ -318,8 +318,7 @@ function -(dt1::DateTime, dt2::AbstractCFDateTime)
318318
end
319319

320320
-(dt::AbstractCFDateTime, Δ::Period) = dt + (-Δ)
321-
-(dt::AbstractCFDateTime, Δ::Dates.CompoundPeriod) = dt + (-Δ)
322-
-(dt::AbstractCFDateTime, Δ) = dt + (-Δ)
321+
-(dt::AbstractCFDateTime, Δ::Dates.Period) = dt + (-Δ)
323322

324323
function ==(dt1::AbstractCFDateTime, dt2::AbstractCFDateTime)
325324
return Dates.value(dt1 - dt2) == 0

test/test_operators.jl

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -20,19 +20,24 @@ dt = DateTimeNoLeap(1959, 12, 31, 23, 39, 59, 123)
2020

2121

2222
# difference between time instances
23-
@test DateTimeStandard(2000, 01, 02) - DateTimeStandard(2000, 01, 01) == Dates.Day(1)
23+
@test DateTimeStandard(2000, 1, 2) - DateTimeStandard(2000, 1, 1) == Dates.Day(1)
2424

25-
@test DateTimeStandard(2000, 01, 02) - DateTimeStandard(2000, 01, 01, units = :day) == Dates.Day(1)
25+
@test DateTimeStandard(2000, 1, 2) - DateTimeStandard(2000, 1, 1, units = :day) == Dates.Day(1)
2626

27-
@test CFTime.units(DateTimeStandard(2000, 01, 01, units = :day, origin = (1970, 1, 1))) == "days since 1970-01-01"
27+
@test (
28+
DateTimeStandard(2000, 1, 2) -
29+
DateTimeStandard(2000, 1, 1, origin = (1970, 1, 1))
30+
) == Dates.Day(1)
31+
32+
@test CFTime.units(DateTimeStandard(2000, 1, 1, units = :day, origin = (1970, 1, 1))) == "days since 1970-01-01"
2833

2934
# check ordering
3035

31-
@test DateTimeStandard(2000, 01, 01) < DateTimeStandard(2000, 01, 02)
32-
@test DateTimeStandard(2000, 01, 01) DateTimeStandard(2000, 01, 01)
36+
@test DateTimeStandard(2000, 1, 1) < DateTimeStandard(2000, 1, 2)
37+
@test DateTimeStandard(2000, 1, 1) DateTimeStandard(2000, 1, 1)
3338

34-
@test DateTimeStandard(2000, 01, 03) > DateTimeStandard(2000, 01, 02)
35-
@test DateTimeStandard(2000, 01, 03) DateTimeStandard(2000, 01, 01)
39+
@test DateTimeStandard(2000, 1, 03) > DateTimeStandard(2000, 1, 2)
40+
@test DateTimeStandard(2000, 1, 03) DateTimeStandard(2000, 1, 1)
3641

3742
# issue #55
3843

0 commit comments

Comments
 (0)