Skip to content

Commit 6349af5

Browse files
committed
Remove unnecessary compare implementations
1 parent 0a3aea5 commit 6349af5

File tree

3 files changed

+21
-12
lines changed

3 files changed

+21
-12
lines changed

src/comparison.jl

Lines changed: 16 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -311,18 +311,16 @@ Base.@pure function Base.cmp(v1::AbstractVariable, v2::AbstractVariable)
311311
return -cmp(name(v1), name(v2))
312312
end
313313

314-
function Base.cmp(m1::AbstractMonomial, m2::AbstractMonomial)
315-
s1, s2 = promote_variables(m1, m2)
316-
return cmp(ordering(m1)(), exponents(s1), exponents(s2))
314+
function Base.:(==)(m1::AbstractMonomialLike, m2::AbstractMonomialLike)
315+
return iszero(cmp(m1, m2))
317316
end
318317

319-
function compare(
318+
function Base.cmp(
320319
m1::AbstractMonomial,
321320
m2::AbstractMonomial,
322-
::Type{O},
323-
) where {O<:AbstractMonomialOrdering}
321+
)
324322
s1, s2 = promote_variables(m1, m2)
325-
return cmp(O(), exponents(s1), exponents(s2))
323+
return cmp(ordering(m1)(), exponents(s1), exponents(s2))
326324
end
327325

328326
# Implement this to make coefficients be compared with terms.
@@ -336,15 +334,26 @@ end
336334
# less than `b`, they are considered sort of equal.
337335
_cmp_coefficient(a, b) = 0
338336

337+
<<<<<<< HEAD
339338
function Base.cmp(t1::AbstractTermLike, t2::AbstractTermLike)
339+
=======
340+
function Base.cmp(
341+
t1::AbstractTermLike,
342+
t2::AbstractTermLike,
343+
)
344+
>>>>>>> e84a75b (Remove unnecessary compare implementations)
340345
Δ = cmp(monomial(t1), monomial(t2))
341346
if iszero(Δ)
342347
return _cmp_coefficient(coefficient(t1), coefficient(t2))
343348
end
344349
return Δ
345350
end
346351

352+
<<<<<<< HEAD
347353
Base.isless(t1::AbstractTermLike, t2::AbstractTermLike) = compare(t1, t2) < 0
354+
=======
355+
Base.isless(t1::AbstractTermLike, t2::AbstractTermLike) = cmp(t1, t2) < 0
356+
>>>>>>> e84a75b (Remove unnecessary compare implementations)
348357

349358
"""
350359
struct ExponentsIterator{M}(

src/default_polynomial.jl

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ Base.one(p::Polynomial) = one(typeof(p))
9393
Base.zero(::Type{Polynomial{C,T,A}}) where {C,T,A} = Polynomial{C,T,A}(A())
9494
Base.zero(t::Polynomial) = zero(typeof(t))
9595

96-
compare_monomials(a, b) = compare(monomial(a), monomial(b))
96+
compare_monomials(a, b) = cmp(monomial(a), monomial(b))
9797

9898
function join_terms(
9999
terms1::AbstractArray{<:Term},
@@ -294,7 +294,7 @@ function __polynomial_merge!(
294294
if tp isa Int && j isa Int
295295
tp = get1(tp)
296296
end
297-
compare(monomial(*(monomials(q)[j], monomial.(t)...)), monomial(tp))
297+
cmp(monomial(*(monomials(q)[j], monomial.(t)...)), monomial(tp))
298298
end
299299
end
300300
get2 = let t = t, q = q
@@ -385,7 +385,7 @@ function __polynomial_merge!(
385385
if tp isa Int && j isa Int
386386
tp = get1(tp)
387387
end
388-
compare(monomial(monomial(t) * monomials(q)[j]), monomial(tp))
388+
cmp(monomial(monomial(t) * monomials(q)[j]), monomial(tp))
389389
end
390390
end
391391
get2 = let t = t, q = q
@@ -460,7 +460,7 @@ function __polynomial_merge!(
460460
if t isa Int && j isa Int
461461
t = get1(t)
462462
end
463-
compare(monomials(q)[j], monomial(t))
463+
cmp(monomials(q)[j], monomial(t))
464464
end
465465
end
466466
get2 = let q = q

src/sequences.jl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ function merge_sorted!(
1818
while i1 <= lastindex(v1) && i2 <= lastindex(v2)
1919
x1 = v1[i1]
2020
x2 = v2[i2]
21-
Δ = compare(x1, x2)
21+
Δ = cmp(x1, x2)
2222
if Δ < 0
2323
if filter(x1)
2424
result[i] = x1

0 commit comments

Comments
 (0)