prefer pre-increment

frederic-mahe · frederic-mahe · commit 59d728ddca04 · 2025-08-27T17:17:10.000+02:00
diff --git a/src/minheap.cc b/src/minheap.cc
@@ -207,7 +207,8 @@ auto minheap_add(minheap_t * a_minheap, elem_t * n) -> void
     {
       /* space for another item at end; swap upwards */
 
-      int index = a_minheap->count++;
+      int index = a_minheap->count;
+      ++a_minheap->count;
       int pos = (index - 1) / 2;
       while ((index > 0) && (elem_smaller(n, a_minheap->array + pos) != 0))
         {
diff --git a/src/search.cc b/src/search.cc
@@ -431,12 +431,12 @@ auto search_thread_run(int64_t t) -> void
           xpthread_mutex_lock(&mutex_output);
 
           /* update stats */
-          queries++;
+          ++queries;
           queries_abundance += qsize;
 
           if (match != 0)
             {
-              qmatches++;
+              ++qmatches;
               qmatches_abundance += qsize;
             }
 

Original file line number	Diff line number	Diff line change
`@@ -207,7 +207,8 @@ auto minheap_add(minheap_t * a_minheap, elem_t * n) -> void`
`207`	`207`	`{`
`208`	`208`	`/* space for another item at end; swap upwards */`
`209`	`209`
`210`		`- int index = a_minheap->count++;`
	`210`	`+ int index = a_minheap->count;`
	`211`	`+ ++a_minheap->count;`
`211`	`212`	`int pos = (index - 1) / 2;`
`212`	`213`	`while ((index > 0) && (elem_smaller(n, a_minheap->array + pos) != 0))`
`213`	`214`	`{`
Original file line number	Diff line number	Diff line change
`@@ -431,12 +431,12 @@ auto search_thread_run(int64_t t) -> void`
`431`	`431`	`xpthread_mutex_lock(&mutex_output);`
`432`	`432`
`433`	`433`	`/* update stats */`
`434`		`- queries++;`
	`434`	`+ ++queries;`
`435`	`435`	`queries_abundance += qsize;`
`436`	`436`
`437`	`437`	`if (match != 0)`
`438`	`438`	`{`
`439`		`- qmatches++;`
	`439`	`+ ++qmatches;`
`440`	`440`	`qmatches_abundance += qsize;`
`441`	`441`	`}`
`442`	`442`