wp/Marlin-2-0-x-Anycubic-i3-MEGA-S
1
0
Fork 0
Code Issues 28 Pull Requests 2 Actions Packages Projects Releases 30 Wiki Activity

Use PROGMEM for menu entries to save RAM #82

Merged
chschu merged 1 commits from feature/optimize-menu-entries into master 2021-04-10 17:09:56 -05:00
Conversation 1 Commits 1 Files Changed 1 +66 -66
chschu commented 2021-02-14 08:58:31 -06:00 (Migrated from github.com)
Copy Link

Description

This PR uses PROGMEM for the LCD menu entries, saving a lot of RAM.

When building for env:MEGA_X from the current master:

RAM:   [========= ]  85.1% (used 6968 bytes from 8192 bytes)
Flash: [=====     ]  52.0% (used 132074 bytes from 253952 bytes)

With the changes from this PR:

RAM:   [========  ]  76.2% (used 6242 bytes from 8192 bytes)
Flash: [=====     ]  51.7% (used 131362 bytes from 253952 bytes)

Benefits

Freeing RAM allows increasing buffer sizes or adding features without risking a stack overflow.

Configurations

Any configuration that uses AnycubicTouchscreenClass.

Related Issues

None that I know of.

### Description This PR uses PROGMEM for the LCD menu entries, saving a lot of RAM. When building for env:MEGA_X from the current master: ``` RAM: [========= ] 85.1% (used 6968 bytes from 8192 bytes) Flash: [===== ] 52.0% (used 132074 bytes from 253952 bytes) ``` With the changes from this PR: ``` RAM: [======== ] 76.2% (used 6242 bytes from 8192 bytes) Flash: [===== ] 51.7% (used 131362 bytes from 253952 bytes) ``` ### Benefits Freeing RAM allows increasing buffer sizes or adding features without risking a stack overflow. ### Configurations Any configuration that uses AnycubicTouchscreenClass. ### Related Issues None that I know of.
knutwurst commented 2021-04-10 17:10:14 -05:00 (Migrated from github.com)
Copy Link

Ahh.. that's a great Idea! Thank you very much!

Ahh.. that's a great Idea! Thank you very much!
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something in the Firmware isn't working
documentation
Improvements or additions to documentation
duplicate
This issue or pull request already exists
good first issue
Good for newcomers
help wanted
Extra attention is needed
invalid
This doesn't seem right
new feature
New feature or request
question
Further information is requested
stale-closing-soon
wontfix
This will not be worked on
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
wp
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: wp/Marlin-2-0-x-Anycubic-i3-MEGA-S#82
No description provided.
Delete Branch "feature/optimize-menu-entries"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?