Mercurial > ~darius > hgwebdir.cgi > stm32temp
diff tempctrl.c @ 43:b4440f9f6d47
Fix 'none' mode.
author | Daniel O'Connor <darius@dons.net.au> |
---|---|
date | Mon, 01 Apr 2013 23:38:34 +1030 |
parents | 56b32b0c9a40 |
children | adc9b1555f9d |
line wrap: on
line diff
--- a/tempctrl.c Mon Apr 01 21:12:57 2013 +1030 +++ b/tempctrl.c Mon Apr 01 23:38:34 2013 +1030 @@ -175,7 +175,7 @@ /* Don't do any logging, just force idle and leave */ if (settings.mode == TC_MODE_NOTHING) { nextstate = 'i'; - goto setstate; + goto skip; } /* Update our temperatures @@ -208,7 +208,7 @@ switch (currstate) { case 'i': /* If we're idle then only heat or cool if the temperate difference is out of the - * hysteresis band + * hysteresis band and the heater/cooler have been off long enough */ if (abs(diff) > settings.hysteresis) { if (diff < 0 && settings.minheatofftime + lastheatoff < t) @@ -254,7 +254,8 @@ else if (settings.mode == TC_MODE_COOL) nextstate = 'c'; - // Keep track of when we last turned things on or off + /* Keep track of when we last turned things on or off */ + skip: switch (nextstate) { case 'c': if (currstate == 'h') @@ -279,18 +280,19 @@ currstate = nextstate; - localtime_r(&t, &tm); - assert(strftime(buf, sizeof(buf) - 1, "%Y/%m/%d %H:%M:%S: ", &tm) != 0); - fputs(buf, stdout); - printtemp("Tr", settings.target_temp, ", "); - printtemp("Fm", tempt, ", "); // Use actual value from sensor - printtemp("Fr", fridge_temp, ", "); - printtemp("Am", ambient_temp, ", "); - printf("St: %s, Fl: %s%s\r\n", state2long(currstate), - forced ? "F" : "", - stale ? "S" : ""); + if (settings.mode != TC_MODE_NOTHING) { + localtime_r(&t, &tm); + assert(strftime(buf, sizeof(buf) - 1, "%Y/%m/%d %H:%M:%S: ", &tm) != 0); + fputs(buf, stdout); + printtemp("Tr", settings.target_temp, ", "); + printtemp("Fm", tempt, ", "); // Use actual value from sensor + printtemp("Fr", fridge_temp, ", "); + printtemp("Am", ambient_temp, ", "); + printf("St: %s, Fl: %s%s\r\n", state2long(currstate), + forced ? "F" : "", + stale ? "S" : ""); + } - setstate: setstate(currstate); }