Skip to content

Commit db41d90

Browse files
committed
DateTimeController: make possible functions const
MonthShortToString and DayOfWeekShortToString don't change the underlying object. Those are just getters and can be declared `const`.
1 parent 21da586 commit db41d90

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

src/components/datetime/DateTimeController.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -108,11 +108,11 @@ void DateTime::UpdateTime(uint32_t systickCounter) {
108108
}
109109
}
110110

111-
const char* DateTime::MonthShortToString() {
111+
const char* DateTime::MonthShortToString() const {
112112
return MonthsString[static_cast<uint8_t>(month)];
113113
}
114114

115-
const char* DateTime::DayOfWeekShortToString() {
115+
const char* DateTime::DayOfWeekShortToString() const {
116116
return DaysStringShort[static_cast<uint8_t>(dayOfWeek)];
117117
}
118118

src/components/datetime/DateTimeController.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,8 +61,8 @@ namespace Pinetime {
6161
return second;
6262
}
6363

64-
const char* MonthShortToString();
65-
const char* DayOfWeekShortToString();
64+
const char* MonthShortToString() const;
65+
const char* DayOfWeekShortToString() const;
6666
static const char* MonthShortToStringLow(Months month);
6767

6868
std::chrono::time_point<std::chrono::system_clock, std::chrono::nanoseconds> CurrentDateTime() const {

0 commit comments

Comments
 (0)