minor improvements

Author: dkarrasch

src/blockmap.jl

@@ -25,7 +25,7 @@ BlockMap{T}(maps::As, rows::Rs) where {T, As<:LinearMapTuple, Rs} =
 MulStyle(A::BlockMap) = MulStyle(A.maps...)
 
 @inline function _getranges(maps, dim, inds::NTuple{N,Int}=ntuple(identity, Val(length(maps)))) where {N}
-    sizes = ntuple(i -> (@inbounds size(maps[inds[i]], dim)), Val(length(inds)))
+    sizes = ntuple(i -> (@inbounds size(maps[inds[i]], dim)), Val(N))
     ends = cumsum(sizes)
     starts = (1, (1 .+ Base.front(ends))...)
     return UnitRange.(starts, ends)
@@ -49,9 +49,9 @@ function rowcolranges(maps, rows)
     # introduce "line breaks"
     colranges = ntuple(Val(length(maps))) do i
         # for each map find the index of the respective row-wise first map
-        firstmapind = firstmapinds[something(findlast(<=(i), firstmapinds), 1)]
+        @inbounds firstmapind = firstmapinds[something(findlast(<=(i), firstmapinds), 1)]
         # shift ranges by the first col-index of the row-wise first map
-        return temp[i] .- first(temp[firstmapind]) .+ 1
+        return @inbounds temp[i] .- first(temp[firstmapind]) .+ 1
     end
     return rowranges, colranges
 end
@@ -88,7 +88,7 @@ function Base.hcat(As::Union{LinearMap, UniformScaling, AbstractVecOrMat}...)
     j = findfirst(x -> !isa(x, UniformScaling), As)
     # this should not happen, function should only be called with at least one LinearMap
     @assert !isnothing(j)
-    nrows = size(As[j], 1)
+    @inbounds nrows = size(As[j], 1)
 
     return BlockMap{T}(promote_to_lmaps(ntuple(_ -> nrows, Val(nbc)), 1, 1, As...), (nbc,))
 end
@@ -126,9 +126,9 @@ function Base.vcat(As::Union{LinearMap,UniformScaling,AbstractVecOrMat}...)
     j = findfirst(x -> !isa(x, UniformScaling), As)
     # this should not happen, function should only be called with at least one LinearMap
     @assert !isnothing(j)
-    ncols = size(As[j], 2)
+    @inbounds ncols = size(As[j], 2)
 
-    rows = ntuple(i->1, Val(nbr))
+    rows = ntuple(_ -> 1, Val(nbr))
     return BlockMap{T}(promote_to_lmaps(ntuple(_ -> ncols, Val(nbr)), 1, 2, As...), rows)
 end