# HG changeset patch
# User Jan Mulder <jlmulder@xs4all.nl>
# Date 1556795297 -7200
# Node ID 04cdeff802544d339480441546b3ba0b5779cefe
# Parent  c7e665e0b08f38c16f52095a90806ee346bfb1d5
Bugfix: write NDL in logbook correctly

This is a subtle one. Typecasting takes precedence over, for example, division.
So, we first typecasted an int to an uint8_t, loosing data in the process, and
then do a seconds to minute conversion. This, obviously, does not give the
result one expects.

And in hindsight a trivial fix for a bug that like has been there forever.

Signed-off-by: Jan Mulder <jlmulder@xs4all.nl>

diff -r c7e665e0b08f -r 04cdeff80254 Discovery/Src/logbook.c
--- a/Discovery/Src/logbook.c	Wed May 01 14:45:47 2019 +0200
+++ b/Discovery/Src/logbook.c	Thu May 02 13:08:17 2019 +0200
@@ -520,7 +520,7 @@
 				{
 					sample[length] = 0;
 					length += 1;
-					sample[length] = (uint8_t)pDecoinfo->output_ndl_seconds / 60;
+					sample[length] = (uint8_t)(pDecoinfo->output_ndl_seconds / 60);
 
 					// Limit stored sample within 0 to 240 mins (Since it's 8bit UINT only)
 					if ((pDecoinfo->output_ndl_seconds / 60) > 240) sample[length] = 240;