Skip to content

Commit fa44a13

Browse files
committed
fix ambiguity warnings
1 parent 5f06b2b commit fa44a13

File tree

1 file changed

+12
-11
lines changed

1 file changed

+12
-11
lines changed

src/DistributedArrays.jl

Lines changed: 12 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -496,13 +496,15 @@ Base.reduce(f, d::DArray) =
496496
mapreduce(fetch, f,
497497
Any[ @spawnat p reduce(f, localpart(d)) for p in procs(d) ])
498498

499-
Base.mapreduce(f, opt::Function, d::DArray) = mapreduce(f, Base.specialized_binary(opt), d)
500-
501-
Base.mapreduce(f, opt, d::DArray) =
499+
function _mapreduce(f, opt, d::DArray)
502500
mapreduce(fetch, opt,
503501
Any[ @spawnat p mapreduce(f, opt, localpart(d)) for p in procs(d) ])
502+
end
503+
Base.mapreduce(f, opt::Union{Base.OrFun,Base.AndFun}, d::DArray) = _mapreduce(f, opt, d)
504+
Base.mapreduce(f, opt::Function, d::DArray) = _mapreduce(f, Base.specialized_binary(opt), d)
505+
Base.mapreduce(f, opt, d::DArray) = _mapreduce(f, opt, d)
504506

505-
Base.map!(f, d::DArray) = begin
507+
Base.map!(f, d::DArray) = begin
506508
@sync for p in procs(d)
507509
@spawnat p map!(f, localpart(d))
508510
end
@@ -579,27 +581,26 @@ for (fn, fr) in ((:sum, :AddFun),
579581
(:minimum, :MinFun),
580582
(:any, :OrFun),
581583
(:all, :AndFun))
582-
@eval begin
583-
(Base.$fn)(d::DArray) = reduce((Base.$fr)(), d)
584-
end
584+
@eval (Base.$fn)(d::DArray) = reduce((Base.$fr)(), d)
585585
end
586586

587587
# mapreduce like
588588
for (fn, fr1, fr2) in ((:maxabs, :AbsFun, :MaxFun),
589589
(:minabs, :AbsFun, :MinFun),
590590
(:sumabs, :AbsFun, :AddFun),
591591
(:sumabs2, :Abs2Fun, :AddFun))
592-
@eval begin
593-
(Base.$fn)(d::DArray) = mapreduce((Base.$fr1)(), (Base.$fr2)(), d)
594-
end
592+
@eval (Base.$fn)(d::DArray) = mapreduce((Base.$fr1)(), (Base.$fr2)(), d)
595593
end
596594

597595
# semi mapreduce
598596
for (fn, fr) in ((:any, :OrFun),
599597
(:all, :AndFun),
600598
(:count, :AddFun))
601599
@eval begin
602-
(Base.$fn)(f::Union(Base.Callable,Base.Func{1}), d::DArray) = mapreduce(f, (Base.$fr)(), d)
600+
(Base.$fn)(f::Base.IdFun, d::DArray) = mapreduce(f, (Base.$fr)(), d)
601+
(Base.$fn)(f::Base.Predicate, d::DArray) = mapreduce(r, (Base.$fr)(), d)
602+
(Base.$fn)(f::Base.Func{1}, d::DArray) = mapreduce(f, (Base.$fr)(), d)
603+
(Base.$fn)(f::Callable, d::DArray) = mapreduce(f, (Base.$fr)(), d)
603604
end
604605
end
605606

0 commit comments

Comments
 (0)