]> granicus.if.org Git - graphviz/commitdiff
sparse: remove unused 'what_to_sum' from 'SparseMatrix_sum_repeat_entries'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Mon, 23 May 2022 14:39:23 +0000 (07:39 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Wed, 24 Aug 2022 14:06:40 +0000 (07:06 -0700)
This function is never called with `what_to_sum` equal to `SUM_REPEATED_NONE`.

lib/sparse/SparseMatrix.c
lib/sparse/SparseMatrix.h

index 730c3a7a5a418f1179f636136f167430c7a9c899..054d11fac753330d2bf205514c2969cbb7b874d9 100644 (file)
@@ -851,7 +851,7 @@ static SparseMatrix SparseMatrix_from_coordinate_arrays_internal(int nz, int m,
 
 
 
-  if(sum_repeated) A = SparseMatrix_sum_repeat_entries(A, sum_repeated);
+  if(sum_repeated) A = SparseMatrix_sum_repeat_entries(A);
  
   return A;
 }
@@ -1349,13 +1349,11 @@ SparseMatrix SparseMatrix_multiply3(SparseMatrix A, SparseMatrix B, SparseMatrix
   return D;
 }
 
-SparseMatrix SparseMatrix_sum_repeat_entries(SparseMatrix A, int what_to_sum){
+SparseMatrix SparseMatrix_sum_repeat_entries(SparseMatrix A){
   /* sum repeated entries in the same row, i.e., {1,1}->1, {1,1}->2 becomes {1,1}->3 */
   int *ia = A->ia, *ja = A->ja, type = A->type, n = A->n;
   int *mask = NULL, nz = 0, i, j, sta;
 
-  if (what_to_sum == SUM_REPEATED_NONE) return A;
-
   mask = MALLOC(sizeof(int)*((size_t)n));
   for (i = 0; i < n; i++) mask[i] = -1;
 
index 28d5ae1c9bd72265bd5df717f99f85b651615ddc..7f97c0b27ae789cd7e37b206ed61f4de029455fa 100644 (file)
@@ -67,7 +67,7 @@ SparseMatrix SparseMatrix_multiply(SparseMatrix A, SparseMatrix B);
 SparseMatrix SparseMatrix_multiply3(SparseMatrix A, SparseMatrix B, SparseMatrix C);
 
 enum {SUM_REPEATED_NONE = 0, SUM_REPEATED_ALL, };
-SparseMatrix SparseMatrix_sum_repeat_entries(SparseMatrix A, int what_to_sum);
+SparseMatrix SparseMatrix_sum_repeat_entries(SparseMatrix A);
 SparseMatrix SparseMatrix_coordinate_form_add_entry(SparseMatrix A, int irn,
                                                     int jcn, void *val);
 int SparseMatrix_is_symmetric(SparseMatrix A, bool test_pattern_symmetry_only);