[mlpack-svn] r10807 - mlpack/trunk/src/mlpack/core/arma_extend/sparse

[fastlab-svn at coffeetalk-1.cc.gatech.edu]

Author: rcurtin
Date: 2011-12-14 17:07:06 -0500 (Wed, 14 Dec 2011)
New Revision: 10807

Modified:
   mlpack/trunk/src/mlpack/core/arma_extend/sparse/SpSubview_meat.hpp
Log:
Fix a bug which was overwriting memory and may be causing that segfault I am
trying to hunt down on the build server...


Modified: mlpack/trunk/src/mlpack/core/arma_extend/sparse/SpSubview_meat.hpp
===================================================================
--- mlpack/trunk/src/mlpack/core/arma_extend/sparse/SpSubview_meat.hpp	2011-12-14 21:23:39 UTC (rev 10806)
+++ mlpack/trunk/src/mlpack/core/arma_extend/sparse/SpSubview_meat.hpp	2011-12-14 22:07:06 UTC (rev 10807)
@@ -166,12 +166,14 @@
 
   const uword start_col = aux_col1;
   const uword end_col   = aux_col1 + n_cols;
+  const uword start_row = aux_row1;
+  const uword end_row   = aux_row1 + n_rows;
 
   for(uword c = start_col; c < end_col; ++c)
     {
-    for(uword r = m.col_ptrs[c]; r < m.col_ptrs[++c]; ++r)
+    for(uword r = start_row; r < end_row; ++r)
       {
-      access::rw(m).values[r] /= val;
+      access::rw(m).at(r, c) /= val;
       }
     }
   }