Skip to content

Use ints instead of enum class to identify touch buttons #56

Open
@alranel

Description

@alranel

As of now, touch buttons are identified using an enum class:

typedef enum {
TOUCH0 = 0,
TOUCH1,
TOUCH2,
TOUCH3,
TOUCH4
} touchButtons;

This means that in order to iterate over all the buttons to find out which one was pressed, a cast to the touchButtons type must be used:

for (int i = 0; i <= 4; ++i) {
   if (carrier.Buttons.onTouchDown((touchButtons)i)) {
      Serial.print("Button pressed: ");
      Serial.println(i);
   }
}

Considering this type is not documented anywhere and probably not strictly necessary, can't we just allow plan ints as arguments and get rid of those TOUCHx constants?

Metadata

Metadata

Assignees

Labels

topic: codeRelated to content of the project itselftype: enhancementProposed improvement

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions