WLED icon indicating copy to clipboard operation
WLED copied to clipboard

Wled disable more

Open FreezyExp opened this issue 2 years ago • 0 comments

I shaved off another 20 KB or so by removing IR support and by adding more WLED_DISABLED wrappings around various files. It gets kind of messy, but it removes unused structs, includes, lopping off entire blobs.

It would be nice if WLED_DISABLE was not paired with WLED_ENABLE so inconsistently, a more thorough cleanup is required.

The fcn_declare.h file was a great starting point for breaking and then fixing all related calls. Perhaps this file could be changed into a module wrapper, which then includes the files needed for a certain functionality.

Those files could include a header per-plugin

#include "wled.h"

#ifndef WLED_DISABLE_X // includes needed for this module // variables // structs // function & macro declarations in the order of: // functions used from wled // functions used within the module // functions used by other modules #endif

this would fascilitate a thorough cleanup of all the #ifdef / #ifndef wrappers by replacing those with empty macros instead of empty functions

And make it a lot easier to track all the various structs / pointers / counters / variables needed for all these modules. As those would all be moved into the module and would no longer exist in various other files.

Aside from this commit though, I currently have no time to go through the steps of this major refactoring of the code to clean up the project.

FreezyExp avatar Dec 28 '22 14:12 FreezyExp