philips-software/amp-embedded-infra-lib

PartitionedTime constructor has incorrect initialization order of its members

daantimmer opened this issue · 0 comments

// source:
        : years(years)
        , months(months)
        , days(days)
        , hours(hours)
        , minutes(minutes)
        , seconds(seconds)
// header:
        uint8_t seconds; // seconds after the minute - [0, 59]
        uint8_t minutes; // minutes after the hour - [0, 59]
        uint8_t hours;   // hours since midnight - [0, 23]
        uint8_t days;    // day of the month - [1, 31]
        uint8_t months;  // months since January - [1, 12]
        uint16_t years;  // years since year 0

Note: -Wextra was required to for this warning to popup

Fixed in #34