Skip to content

Commit 94aefed

Browse files
Riksu9000JF002
authored andcommitted
Condense firmware validation code
1 parent 217f16d commit 94aefed

2 files changed

Lines changed: 10 additions & 28 deletions

File tree

src/displayapp/screens/FirmwareValidation.cpp

Lines changed: 9 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -16,30 +16,17 @@ namespace {
1616

1717
FirmwareValidation::FirmwareValidation(Pinetime::Applications::DisplayApp* app, Pinetime::Controllers::FirmwareValidator& validator)
1818
: Screen {app}, validator {validator} {
19-
labelVersionInfo = lv_label_create(lv_scr_act(), nullptr);
20-
lv_obj_align(labelVersionInfo, nullptr, LV_ALIGN_IN_TOP_LEFT, 0, 0);
21-
lv_label_set_text(labelVersionInfo, "Version : ");
22-
lv_label_set_align(labelVersionInfo, LV_LABEL_ALIGN_LEFT);
23-
24-
labelVersionValue = lv_label_create(lv_scr_act(), nullptr);
25-
lv_obj_align(labelVersionValue, labelVersionInfo, LV_ALIGN_OUT_RIGHT_MID, 0, 0);
26-
lv_label_set_recolor(labelVersionValue, true);
27-
sprintf(version, "%ld.%ld.%ld", Version::Major(), Version::Minor(), Version::Patch());
28-
lv_label_set_text(labelVersionValue, version);
29-
30-
labelShortRefInfo = lv_label_create(lv_scr_act(), nullptr);
31-
lv_obj_align(labelShortRefInfo, nullptr, LV_ALIGN_IN_TOP_LEFT, 0, 25);
32-
lv_label_set_text(labelShortRefInfo, "ShortRef : ");
33-
lv_label_set_align(labelShortRefInfo, LV_LABEL_ALIGN_LEFT);
34-
35-
labelShortRefValue = lv_label_create(lv_scr_act(), nullptr);
36-
lv_obj_align(labelShortRefValue, labelShortRefInfo, LV_ALIGN_OUT_RIGHT_MID, 0, 0);
37-
lv_label_set_recolor(labelShortRefValue, true);
38-
sprintf(shortref, "%s", Version::GitCommitHash());
39-
lv_label_set_text(labelShortRefValue, shortref);
19+
labelVersion = lv_label_create(lv_scr_act(), nullptr);
20+
lv_label_set_text_fmt(labelVersion, "Version : %d.%d.%d\n"
21+
"ShortRef : %s",
22+
Version::Major(),
23+
Version::Minor(),
24+
Version::Patch(),
25+
Version::GitCommitHash());
26+
lv_obj_align(labelVersion, nullptr, LV_ALIGN_IN_TOP_LEFT, 0, 0);
4027

4128
labelIsValidated = lv_label_create(lv_scr_act(), nullptr);
42-
lv_obj_align(labelIsValidated, nullptr, LV_ALIGN_IN_TOP_LEFT, 0, 50);
29+
lv_obj_align(labelIsValidated, labelVersion, LV_ALIGN_OUT_BOTTOM_LEFT, 0, 0);
4330
lv_label_set_recolor(labelIsValidated, true);
4431
lv_label_set_long_mode(labelIsValidated, LV_LABEL_LONG_BREAK);
4532
lv_obj_set_width(labelIsValidated, 240);

src/displayapp/screens/FirmwareValidation.h

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,7 @@ namespace Pinetime {
2323
private:
2424
Pinetime::Controllers::FirmwareValidator& validator;
2525

26-
lv_obj_t* labelVersionInfo;
27-
lv_obj_t* labelVersionValue;
28-
lv_obj_t* labelShortRefInfo;
29-
lv_obj_t* labelShortRefValue;
30-
char version[9];
31-
char shortref[9];
26+
lv_obj_t* labelVersion;
3227
lv_obj_t* labelIsValidated;
3328
lv_obj_t* buttonValidate;
3429
lv_obj_t* labelButtonValidate;

0 commit comments

Comments
 (0)