- May 23, 2016
-
-
Max Horn authored
-
- May 05, 2016
-
-
Max Horn authored
The old (and new again) semantics are that the bits in 'data' are sent starting from the MSB. This had regressed recently to sending bits starting at the LSB, as part of my refactoring. Should resolve #52
-
- May 04, 2016
-
-
Max Horn authored
-
Max Horn authored
-
Max Horn authored
Previously, for each bit we transmitted, we disabled the receiver interrupt, sent the bit (as a high-low pulse), then re-enabled the interrupt. For a 24bit tristate word, that amounts to disabling and enabling the interrupt 48 times. Now we only do it once.
-
Max Horn authored
-
Max Horn authored
Previously, sending worked by decoding a string which encoded a binary or tristate codeword. Trying to send a bit pattern directly (encoded as an integer) involved turning it into a string first. This is a waste of memory and CPU cycles. So we do it the other way around now: The central send() function accepts an integer containing the bit pattern to be sent. The other send() function which takes a 0-1-string, as well as sendTriState(), now call this new central send() function. Incidentally, this reduces code duplication and enables further refactoring.
-
Max Horn authored
This should fix #46
-
Max Horn authored
-
Max Horn authored
In particular, there is no need to #define CHANGE (it is not used in the code if RaspberryPi is #defined), and don't override PROGMEM and memcpy_P in the header; instead do that in the .c file, to avoid polluting the global namespace
-
Max Horn authored
Removed unnecessary 'extern C'-block
-
- May 03, 2016
-
-
Max Horn authored
-
Max Horn authored
-
Max Horn authored
Since handleInterrupt always sets to changeCount a bit later, this does not change program behaviour. It does, however, make it easier to understand the code, and in particular, to verify that no overflow occurs here. See issue #44.
-
Max Horn authored
This commit mainly adds comments, and gets rid of one else-if, so it should be easy to see that no functionality was changed so far.
-
Max Horn authored
-
Max Horn authored
It is now much simpler, results in a smaller binary, and is more similar to the other getCodeWord methods
-
Max Horn authored
-
Max Horn authored
Also moved it to be before getCodeWordB, so the getCodeWord* methods are now sorted alphabetically.
-
Max Horn authored
-
Max Horn authored
-
Max Horn authored
Overall, the RCSwitch code is a mess when it comes to uniform formatting; but overall, it seems using 2 space indention is predominant, so let's try to make sure new code uses that, and slowly migrate existing code to use it.
-
Max Horn authored
Some compilers rightfully complain about this; it could also fool somebody into thinking a string was returned, when in reality that's not the case.
-
Max Horn authored
The new code is smaller and faster and perhaps also easier to understand
-
- May 02, 2016
- Apr 28, 2016
-
-
Robert ter Vehn authored
-
- Mar 26, 2016
-
-
Max Horn authored
fix 2 bugs introduced in PR #35
-
- Mar 20, 2016
-
-
Vlad Gheorghe authored
-
Vlad Gheorghe authored
-
Vlad Gheorghe authored
Test case: https://gist.github.com/vgheo/f34ac7afd6447b3f1fc4
-
- Mar 11, 2016
-
-
Suat Özgür authored
Iterate over all defined protocols in `handleInterrupt`
-
Suat Özgür authored
Travis integration + Readme changes
-
- Mar 05, 2016
-
-
Kyle Anderson authored
-
Kyle Anderson authored
-
- Mar 01, 2016
-
-
Johann Richard authored
Fixed a dumb error (duplicate lines)
-
johannrichard authored
So far, while decoding the timings, the `handleInterrupt`routine only iterated over protocols 1-3 before failing. This small change will iterate over *all* protocols that have been defined.
-
Suat Özgür authored
Tweak initial high signal of protocol 3
-
- Feb 16, 2016
-
-
Max Horn authored
This resolves issue #32 (with a recent Globaltronics GT-FSI-07), and also is necessary for compatibility with my Brennenstuhl RCS 2044.
-