-
Notifications
You must be signed in to change notification settings - Fork 66
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Get rid of fixed device numbers #253
Merged
Merged
Changes from 1 commit
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
03c36c8
dynamical button numbers
elral 45edfaf
set up array function
elral ce9d55c
better comments
elral 701dcc1
all devices with dymically defined array
elral 088167d
changes for MUX
elral c080f58
missing defines added
elral 34e83fc
missing return statement
elral 1762966
back to original device memory space
elral 03ec932
debug print out added
elral 97ed2d6
extended information of memory usage to serial, deactivated
elral 838888e
clean up, delete max. number of devices from board file
elral f77ebe8
get servos to run
elral dcbb30a
add comment for array size
elral cb038b2
get array size via an array
elral f2b0955
error handling added
elral 666fec8
better definition of array size
elral ea09be0
intended size for nameBuffer[]
elral e3fe451
increased deviceBuffer for AVR's, decreased for Pico
elral 6230b32
Merge branch 'main' into no_fixed_device_numbers
elral 8584aaa
Merge branch 'MobiFlight:main' into no_fixed_device_numbers
elral 517385b
max. numbers of devices not required anymore
elral 63f9c36
one common function for sending failure message
elral 8dd5291
failure message directly from flash
elral 096be82
better naming
elral 27abcd9
delete not required parameter in handler functions
elral 0db4409
Revert "delete not required parameter in handler functions"
elral 50a7526
Merge branch 'main' into no_fixed_device_numbers
elral 018b451
merge branch main
elral 4e8eba1
delete not required folder
elral 8cb5d3b
Merge branch 'main' into no_fixed_device_numbers
elral d2a42d5
Merge branch 'main' into no_fixed_device_numbers
elral daac96f
merge branch main
elral 64403aa
also custom devices w/o device limit
elral 910f7b2
setup array for custom device was missing
elral File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggestion: add a last entry like this (comments removed for clarity)
and use that value for the array size in
getArraysizes()
. This way no manual update will be required when adding devices.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Again a very good idea!