Saving three bytes of memory with this crazy loop structure..
-
Wow, what a response! Amazing stuff!
thanks @mfalkvidd for the explanation.
@Yveaux that millis() / 1000 is very elegant. That's a very interesting direction.
For now though, looking at some of the creations.. I think I'll stick with my modulo system for readability :-D Very cool though.
@Yveaux @mfalkvidd Now it gets ugly - and we are not talking ASM yet... :stuck_out_tongue_winking_eye:
ISR (WDT_vect) { WDTCSR = _BV(WDCE) | _BV(WDE); WDTCSR = _BV(WDIF) | _BV(WDIE) | 6; // 1s EEARL = 1; } void setup() { WDT_vect(); } void loop() { if(EEARL) { EEARL = 0; // ... code that runs every second... } }Flash: 502 bytes
Ram: 9 bytes -
@tekka Whoa :-) Can you elaborate what your code voodoo does a little bit?
For a balance of readability, I was pondering how to make this:
- Divide millis() / 1000
- round that down
- get the last bit of that rounded down variable
- if that last bit is different that before, a second has passed.
-
@tekka Whoa :-) Can you elaborate what your code voodoo does a little bit?
For a balance of readability, I was pondering how to make this:
- Divide millis() / 1000
- round that down
- get the last bit of that rounded down variable
- if that last bit is different that before, a second has passed.
-
@tekka Whoa :-) Can you elaborate what your code voodoo does a little bit?
For a balance of readability, I was pondering how to make this:
- Divide millis() / 1000
- round that down
- get the last bit of that rounded down variable
- if that last bit is different that before, a second has passed.
-
@alowhum Instead of a timer, the watchdog interrupt is used to set a flag on a (in this sketch) not-used eeprom addressing register (EEARL)...certainly not a generic approach, but functional :smiley:
-
@mfalkvidd Strictly speaking, only @Yveaux's solution would work without modifications to the MySensors core :) But the challenge was weakly defined, so no cheating in that sense :sweat_smile:
-
@mfalkvidd Strictly speaking, only @Yveaux's solution would work without modifications to the MySensors core :) But the challenge was weakly defined, so no cheating in that sense :sweat_smile:
-
@mfalkvidd Strictly speaking, only @Yveaux's solution would work without modifications to the MySensors core :) But the challenge was weakly defined, so no cheating in that sense :sweat_smile:
-
@alowhum trading one (or more) resource(s) for another is what optimizing is about. So if you need ram, this is reasonable. And no, using modulus will not affect the performance of most applications.
What I would worry about is the maintainability / readability of the code. What happens if you need to modify the code 2 years from now, will you remember how it works? What is the risk of introducing new bugs or strange side-effects? Is that risk worth saving the ram? If it is, then you use it. I don't see it being anything more complicated than that.
Hi @mfalkvidd,
Problem is, when your code can't compile because you are one (or several) bytes short of ram, nothing else matters.
As to readability, I assume you know about that rarely used compiler feature called 'comments'? I hear they use zero Arduino RAM and even less ROM memory. :flushed:.
Ok, I'm just being cheeky, so don't flame me, it just seemed a good opportunity for a reminder to everybody. Point being we are all guilty of not using enough comments in our code.
You said you are worried about readability and maintainability - it's just like code backups, it's a problem only because we only worry about them 'after' a drive crash, or in the case of comments, two years later when we are trying to remember what the hell this weird code does.. THE REALITY: If we are really worried, we would add liberal comments and do regular backups - otherwise I say we're not really 'that' worried.
Cheers,
Paul
