[PATCH] Show only as many stats as there are DC types

Miika Turkia miika.turkia at gmail.com
Fri Jan 22 12:27:56 PST 2016


After reserving only the required amount of data for stats_by_type, we
showed an extra DC type on the statistics (no more extra space filled
with 0 at the end of the buffer)

Signed-off-by: Miika Turkia <miika.turkia at gmail.com>
---
 qt-models/yearlystatisticsmodel.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/qt-models/yearlystatisticsmodel.cpp b/qt-models/yearlystatisticsmodel.cpp
index 1cc5a90..955f0e8 100644
--- a/qt-models/yearlystatisticsmodel.cpp
+++ b/qt-models/yearlystatisticsmodel.cpp
@@ -206,7 +206,7 @@ void YearlyStatisticsModel::update_yearly_stats()
 	/* Show the statistic sorted by dive type */
 	if (stats_by_type != NULL && stats_by_type[0].selection_size) {
 		YearStatisticsItem *item = new YearStatisticsItem(stats_by_type[0]);
-		for (i = 1; i <= sizeof(dive_comp_type) + 1; ++i) {
+		for (i = 1; i <= NUM_DC_TYPE; ++i) {
 			if (stats_by_type[i].selection_size == 0)
 				continue;
 			YearStatisticsItem *iChild = new YearStatisticsItem(stats_by_type[i]);
-- 
2.5.0



More information about the subsurface mailing list