Merge pull request #33 from fredrikekre/fe/offsetvector

add OffsetVector constructors, and export OffsetVector

Author: alsam

src/OffsetArrays.jl

@@ -5,7 +5,7 @@ module OffsetArrays
 using Base: Indices, tail
 using Compat
 
-export OffsetArray, @unsafe
+export OffsetArray, OffsetVector, @unsafe
 
 # TODO: just use .+
 # See https://github.com/JuliaLang/julia/pull/22932#issuecomment-330711997
@@ -34,6 +34,11 @@ OffsetArray{T}(inds::Vararg{AbstractUnitRange,N}) where {T,N} = OffsetArray{T,N}
 OffsetArray(A::AbstractArray{T,0}) where {T} = OffsetArray{T,0,typeof(A)}(A, ())
 OffsetArray(::Type{T}, inds::Vararg{UnitRange{Int},N}) where {T,N} = OffsetArray{T,N}(inds)
 
+# OffsetVector constructors
+OffsetVector(A::AbstractVector, offset) = OffsetArray(A, offset)
+OffsetVector(::Type{T}, inds::AbstractUnitRange) where {T} = OffsetArray{T,1}(inds)
+OffsetVector{T}(inds::AbstractUnitRange) where {T} = OffsetArray{T}(inds)
+
 # The next two are necessary for ambiguity resolution. Really, the
 # second method should not be necessary.
 OffsetArray(A::AbstractArray{T,0}, inds::Tuple{}) where {T} = OffsetArray{T,0,typeof(A)}(A, ())

test/runtests.jl

@@ -359,3 +359,11 @@ if VERSION >= v"0.7.0-DEV.1790"
     @test summary(a) == "OffsetArray(::Array{$(Int),2}, -1:0, 5:6) with eltype $(Int) with indices -1:0×5:6"
     @test summary(view(a, :, 5)) == "view(OffsetArray(::Array{Int64,2}, -1:0, 5:6), :, 5) with eltype Int64 with indices -1:0"
 end
+
+@testset "OffsetVector constructors" begin
+    v = rand(5)
+    @test OffsetVector(v, -2) == OffsetArray(v, -2)
+    @test OffsetVector(v, -2:2) == OffsetArray(v, -2:2)
+    @test typeof(OffsetVector(Float64, -2:2)) == typeof(OffsetArray(Float64, -2:2))
+    @test typeof(OffsetVector{Float64}(-2:2)) == typeof(OffsetArray{Float64}(-2:2))
+end