Suppose resizedata! on transposed arrays (#392)

Author: DanielVandH

Project.toml

@@ -1,6 +1,6 @@
 name = "LazyArrays"
 uuid = "5078a376-72f3-5289-bfd5-ec5146d43c02"
-version = "2.9.3"
+version = "2.9.4"
 
 [deps]
 ArrayLayouts = "4c555306-a7a7-4459-81d9-ec55ddd5c99a"

src/cache.jl

@@ -228,9 +228,13 @@ resizedata!(B::CachedArray, mn...) = resizedata!(MemoryLayout(B.data), MemoryLay
 resizedata!(B::AbstractCachedArray, mn...) = resizedata!(MemoryLayout(B.data), UnknownLayout(), B, mn...)
 resizedata!(A, mn...) = A # don't do anything
 function resizedata!(A::AdjOrTrans, m, n)
-    m ≤ 0 || resizedata!(parent(A), n)
+    resizedata!(parent(A), n, m)
     A
 end
+function resizedata!(A::AdjOrTrans{<:Any, <:AbstractVector}, m, n)
+    resizedata!(parent(A), n)
+    A
+end 
 
 function cache_filldata!(B, inds...)
     B.data[inds...] .= view(B.array,inds...)

test/cachetests.jl

@@ -538,7 +538,7 @@ using Infinities
         @testset "DualLayout{<:AbstractCachedLayout}" begin
             arg1 = view((1:100)', :, 1:10)
             arg2 = view(AccumulateAbstractVector(*, 1:100)', :, 1:10)
-            bc = Base.Broadcast.Broadcasted(CachedArrayStyle{2}(), *, (arg1, arg2))
+            bc = Base.Broadcast.Broadcasted(CachedArrayStyle{2}(), *, (arg1, arg2));
             rsz_bc = LazyArrays.resize_bcargs!(bc);
             @test rsz_bc.args[2] == view(arg2.parent.parent.data', :, 1:10)
         end
@@ -574,6 +574,13 @@ using Infinities
         @test cacheddata(F) === view(cacheddata(parent(parent(F))), 1:1)'
         @test cacheddata(G) === adjoint(view(cacheddata(parent(G)), 1:1, 1:1))
     end
+
+    @testset "resizedata! for AdjTrans array" begin
+        x = LazyArrays.CachedArray(rand(1, 10))';
+        @test parent(x).datasize == (0, 0)
+        LazyArrays.resizedata!(x, 4, 1)
+        @test parent(x).datasize == (1, 4)
+    end
 end
 
 end # module