merge v0.6.0 into master #8
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ome one-liners to generate them
…fferedEeprom.h to AceUtilsBufferedEepromStm32.h; rename EpoxyProm{Avr,Esp} to EpoxyEeprom{Avr,Esp}
…Interface hierarchy and virtual functions, saving between 150-250 bytes of flash on AVR processors
…zed class, 950 bytes on LedClock and LedClockTiny on AVR
…vious syntax worked on some compiler, broke on others
…ds #include <EEPROM.h>
…hout AceRoutine Channels
…ed_eeprom_stm32/buffered_eeprom_stm32.h
… that reads from a Stream and calls CommandDispatcher directly
…lProcessorManager; add 'Coroutine' to class names which inherit from ace_routine::Coroutine
…d() directly, instead of through a Coroutine
…istency with StreamProcessor* classes
… ChannelProcessorManager
…to Discussions first
…ogether in a single ModeRecord
… to follow and maintain
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
AceUtils/buffered_eeprom_stm32AceUtilsStm32BufferedEeprom.htobuffered_eeprom_stm32/buffered_eeprom_stm32.hAceUtils/crc_eepromCrcEeprom: Recommend using a 32-bit random number for thecontextIdinstead of usingtoContextId()helper function.IEepromAdaptertoEepromInterface;EspEepromAdaptertoEspStyleEeprom;AvrEepromAdaptertoAvrStyleEeprom.EpoxyPromAvrtoEpoxyEepromAvr;EpoxyPromEsptoEpoxyEepromEsp; tracks the changes in EpoxyDuino library.CrcEeprominto a template class, templatized onEspStyleEepromandAvrStyleEeprom. Seems to save between 150-950bytes of flash memory by avoiding virtual function calls and class
hierarchies.
AceUtils/freememfreeMemory()function fromexamples/CommandLineShell.inointo
freemem/freemem.hso that it can be used elsewhere.AceUtils/cliAceUtilsCli.htocli/cli.h.CommandDispatcherfromChannelDispatcherso that it can beused without using an AceRoutine
Channel.CommandManagertoChannelProcessorManager.ChannelDispatchertoChannelProcessorCoroutine.StreamLineReadertoStreamReaderCoroutine.StreamProcessorCoroutineandStreamProcessorManager.DirectProcessorandDirectProcessorManager.examples/CommandLineShell.inointo 3 versions:ChannelCommandLineShellStreamCommandLineShellDirectCommandLineShellAceUtils/mode_groupModeRecordto keep the modeId and itsModeGroup*pointertogether in a single data structure.
because I had a hard time understanding my own code.
notes at the top of src/mode_group. I'll
keep it around for historical reference.