Skip to content

Commit e62df26

Browse files
committed
Drop compat code for logdet(::Any) fallback from #382
1 parent 6d3229e commit e62df26

File tree

4 files changed

+3
-11
lines changed

4 files changed

+3
-11
lines changed

README.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -120,8 +120,6 @@ Currently, the `@compat` macro supports the following syntaxes:
120120

121121
* `isnothing` for testing if a variable is equal to `nothing` ([#29674]).
122122

123-
* `logdet` for `Number`s ([#22629]).
124-
125123
* `fieldcount` is equivalent to `nfields` for Julia versions 0.6 and below and is used to
126124
determine the number of fields in a data type ([#22350]).
127125

src/Compat.jl

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,6 @@ import Base.invokelatest
1414

1515
include("compatmacro.jl")
1616

17-
# https://github.com/JuliaLang/julia/pull/22629
18-
if VERSION < v"0.7.0-DEV.848"
19-
import Base: logdet
20-
logdet(A) = log(det(A))
21-
end
22-
2317
# https://github.com/JuliaLang/julia/pull/22633
2418
if VERSION < v"0.7.0-DEV.1041"
2519
# these have been deprecated in Julia 0.7.0-DEV.5272; we keep them here to avoid

test/old.jl

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -165,3 +165,6 @@ end
165165
@test @inferred(ntuple(x->1, Val(3))) == (1,1,1)
166166
@test @inferred(ntuple(x->x, Val(0))) == ()
167167
@test @inferred(ntuple(x->x, Val(5))) == (1,2,3,4,5)
168+
169+
# PR 22629
170+
@test logdet(0.5) == log(det(0.5))

test/runtests.jl

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -67,9 +67,6 @@ for x in (3.1, -17, 3//4, big(111.1), Inf)
6767
@test minmax(x) == (x, x)
6868
end
6969

70-
# PR 22629
71-
@test logdet(0.5) == log(det(0.5))
72-
7370
# PR 22633
7471
if VERSION < v"0.7.0-DEV.5272"
7572
# chol(A::UniformScaling) has been deprecated in Julia, we still test it to avoid

0 commit comments

Comments
 (0)