mirror of
https://github.com/portapack-mayhem/mayhem-firmware.git
synced 2026-09-10 08:39:29 +00:00
UI and bug fixes (#3164)
* Fix BLE RX RSSI and Channel scale * Properly reset gradient index 0 * Use move constructor in MenuView::add_item * Avoid erasing during iteration * Fix gradient rounding. Extend to end of table * Add comment to add_item
This commit is contained in:
@@ -159,16 +159,15 @@ size_t MenuView::item_count() const {
|
||||
return menu_items.size();
|
||||
}
|
||||
|
||||
void MenuView::add_item(MenuItem new_item) {
|
||||
menu_items.push_back(new_item);
|
||||
void MenuView::add_item(MenuItem&& new_item) {
|
||||
menu_items.push_back(std::move(new_item));
|
||||
|
||||
update_items();
|
||||
}
|
||||
|
||||
void MenuView::add_items(std::initializer_list<MenuItem> new_items) {
|
||||
for (auto item : new_items) {
|
||||
add_item(item);
|
||||
}
|
||||
menu_items.insert(menu_items.end(), new_items);
|
||||
update_items();
|
||||
}
|
||||
|
||||
void MenuView::update_items() {
|
||||
|
||||
@@ -83,7 +83,10 @@ class MenuView : public View {
|
||||
|
||||
~MenuView();
|
||||
|
||||
void add_item(MenuItem new_item);
|
||||
// This function takes a temporary to avoid copies and heap allocations.
|
||||
// To pass a variable 'item', use add_item(std::move(item)),
|
||||
// or construct the MenuItem in the call, add_item(MenuItem{...})
|
||||
void add_item(MenuItem&& new_item);
|
||||
void add_items(std::initializer_list<MenuItem> new_items);
|
||||
void clear();
|
||||
size_t item_count() const;
|
||||
|
||||
Reference in New Issue
Block a user