Check whether ColorView is defined (#67)

ColorView has been deprecated since Nov 2017, but only as a constructor. Hence direct uses of the type did not get noticed. I may "resurrect" `ColorView` in new form, and if so the existing definition here should still work, but in case we go through a period where there is no `ColorView` we should check for its existence.
JuliaIO · Sep 1, 2020 · 42e9aa8 · 42e9aa8 · timholy · Sep 1, 2020
1 parent 333d56e
commit 42e9aa8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/Project.toml b/Project.toml
@@ -1,6 +1,6 @@
 name = "QuartzImageIO"
 uuid = "dca85d43-d64c-5e67-8c65-017450d5d020"
-version = "0.7.2"
+version = "0.7.3"
 
 [deps]
 ColorVectorSpace = "c3611d14-8923-5661-9e6a-0046d554d3a4"

diff --git a/src/QuartzImageIO.jl b/src/QuartzImageIO.jl
@@ -366,8 +366,9 @@ mapCG(x::Normed) = x
 to_contiguous(A::Array) = A
 to_contiguous(A::AbstractArray) = collect(A)
 to_contiguous(A::BitArray) = convert(Array{N0f8}, A)
-to_contiguous(A::ColorView) = to_contiguous(channelview(A))
-
+if isdefined(ImageCore, :ColorView)
+    to_contiguous(A::ColorView) = to_contiguous(channelview(A))
+end
 to_explicit(A::Array{C}) where C <: Colorant = to_explicit(channelview(A))
 to_explicit(A::Base.ReinterpretArray{T}) where T = to_explicit(copyto!(Array{T}(undef, size(A)), A))
 to_explicit(A::Array{T}) where T <: Normed = rawview(A)