Fix compiler error `a label can only be part of a statement and a de…

[LupusE]

…claration is not a statement`

It's only a compiler error, regarding to https://www.educative.io/answers/resolving-the-a-label-can-only-be-part-of-a-statement-error

The change was coming with https://github.com/RfidResearchGroup/ChameleonUltra/pull/314/changes#diff-8e81c0cb796bfec7b1cd15724c4adbeb5bc55bf89909f833de73d29e1ef4a608 and I have no Idea how it compiled before.

[github-actions]

You are welcome to add an entry to the CHANGELOG.md as well

[suut]

You can also enclose the contents of the case within { }, it silences the warning but also has the added effect of not defining the variable globally so you don't end up in that case:

switch (2) {
  case 1:
    int x = 2;
    break;
  case 2: ;
    printf("%d\n", x);
    break;
}

Here it would print a random value for x, it's declared globally because it's part of the switch case in the same scope, but it's not defined because case 1 was not executed, so you end up with a random value.

You have the same kind of problem if you declare variables with the same name in two different cases, they would clash together

So instead you can do

switch (2) {
  case 1: {
    int x = 2;
    break;
  }
  case 2: ;
    printf("%d\n", x);
    break;
}

[github-actions]

Built artifacts for commit 5d2946f

Firmware

• Ultra APP DFU Package
• Ultra binaries
• Lite APP DFU Package
• Lite binaries

Client

• Linux
• macOS
• Windows