Fixed a bug in readRowsBetween (curOutput was already SPIFFSLogData<T>* so didn't need the sizeof())

HMGrunthos · HMGrunthos · commit 3e4609cd262e · 2020-10-21T02:33:04.000+01:00
diff --git a/src/SPIFFSLogger.h b/src/SPIFFSLogger.h
@@ -166,7 +166,7 @@ size_t SPIFFSLogger<T>::readRowsBetween(SPIFFSLogData<T> *output, time_t fromTim
                 hits++;
                 if (hits > startIdx)
                 {
-                    SPIFFSLogData<T> *curOutput = output + (copied * sizeof(SPIFFSLogData<T>));
+                    SPIFFSLogData<T> *curOutput = output + copied;
 
                     // copy the timestamp
                     memcpy(curOutput, &currentTime, sizeof(time_t));

Original file line number	Diff line number	Diff line change
`@@ -166,7 +166,7 @@ size_t SPIFFSLogger<T>::readRowsBetween(SPIFFSLogData<T> *output, time_t fromTim`
`166`	`166`	`hits++;`
`167`	`167`	`if (hits > startIdx)`
`168`	`168`	`{`
`169`		`- SPIFFSLogData<T> curOutput = output + (copied sizeof(SPIFFSLogData<T>));`
	`169`	`+ SPIFFSLogData<T> *curOutput = output + copied;`
`170`	`170`
`171`	`171`	`// copy the timestamp`
`172`	`172`	`memcpy(curOutput, &currentTime, sizeof(time_t));`