fix off-by-one in findn and findnz

``count`` was equal to the number of nonzero elements plus one, so the output vectors had an extra element with junk.
jiahao · Jan 13, 2014 · 5446521 · 5446521
1 parent 5e22c08
commit 5446521
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/base/sparse/sparsematrix.jl b/base/sparse/sparsematrix.jl
@@ -298,7 +298,8 @@ function findn{Tv,Ti}(S::SparseMatrixCSC{Tv,Ti})
  end
  end
 
- if numnz != count-1
+ count -= 1
+ if numnz != count
  I = I[1:count]
  J = J[1:count]
  end
@@ -324,7 +325,8 @@ function findnz{Tv,Ti}(S::SparseMatrixCSC{Tv,Ti})
  end
  end
 
- if numnz != count-1
+ count -= 1
+ if numnz != count
  I = I[1:count]
  J = J[1:count]
  V = V[1:count]