Skip to content

Convert code from .ino to .cpp/.h#3139

Merged
TD-er merged 6 commits intoletscontrolit:megafrom
TD-er:bugfix/ModbusRTU_h_cpp
Jul 3, 2020
Merged

Convert code from .ino to .cpp/.h#3139
TD-er merged 6 commits intoletscontrolit:megafrom
TD-er:bugfix/ModbusRTU_h_cpp

Conversation

@TD-er
Copy link
Member

@TD-er TD-er commented Jun 29, 2020

This is needed to improve build reliability + reduce memory usage since some code was taking up extra space due to the strange way how .ino files are compiled.

Converted:

  • StringConverter.ino
  • Scheduler.ino
  • ESPEasyStorage.ino
  • Hardware.ino
  • ESPEasy_checks.ino
  • Convert.ino

Plus some clean up of now made obsolete forward declarations in ESPEasy_fwddelcs.h

TD-er added 4 commits June 29, 2020 10:51
Also converted: - StringConverter.ino - Scheduler.ino - ESPEasyStorage.ino - Hardware.ino - ESPEasy_checks.ino - Convert.ino Plus some clean up of now made obsolete forward declarations in ESPEasy_fwddelcs.h
@TD-er
Copy link
Member Author

TD-er commented Jun 30, 2020

I made a test build for these changes.
Please let me know if something got broken by these changes.

And also would be nice to see if this resulted in more free memory.

@TD-er
Copy link
Member Author

TD-er commented Jul 3, 2020

Nobody tested it?

@TD-er TD-er merged commit 5ae9038 into letscontrolit:mega Jul 3, 2020
@TD-er TD-er deleted the bugfix/ModbusRTU_h_cpp branch July 3, 2020 12:02
@S51KZ
Copy link

S51KZ commented Jul 3, 2020

1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants