Changeset 3271:d86a714eab75 for sage/matrix/matrix_rational_sparse.pyx

Timestamp:

03/03/07 23:06:15 (6 years ago)

Author:

William Stein <wstein@…>

Branch:

default

Message:

Sparse matrices over the integers; also fixed a bug that prevented sparse matrices over ZZ and QQ from pickling.

File:

• sage/matrix/matrix_rational_sparse.pyx (modified) (2 diffs)

sage/matrix/matrix_rational_sparse.pyx

@@ -38 +38 @@
     ########################################################################
     def __new__(self, parent, entries, copy, coerce):
+        # set the parent, nrows, ncols, etc. 
+        matrix_sparse.Matrix_sparse.__init__(self, parent)
+
         self._matrix = <mpq_vector*> sage_malloc(parent.nrows()*sizeof(mpq_vector))
         if self._matrix == NULL:
             raise MemoryError, "error allocating sparse matrix"
+        # initialize the rows 
+        for i from 0 <= i < parent.nrows():
+            init_mpq_vector(&self._matrix[i], self._ncols, 0)
+
+        # record that rows have been initialized
+        self._initialized = True
+
         
     def __dealloc__(self):
@@ -66 +76 @@
         cdef Rational z
         cdef void** X
-
-        # set the parent, nrows, ncols, etc. 
-        matrix_sparse.Matrix_sparse.__init__(self, parent)
-
-        # initialize the rows 
-        for i from 0 <= i < parent.nrows():
-            init_mpq_vector(&self._matrix[i], self._ncols, 0)
-
-        # record that rows have been initialized
-        self._initialized = True
 
         # fill in entries in the dict case