search

Zanduino / DS3231M

Access the DS3231M I2C Realtime Clock
GNU General Public License v3.0
15 stars 7 forks source link

dayOfTheWeek() has incorrect range #22

Closed Levent-Keskin closed 2 years ago

Levent-Keskin commented 2 years ago

Expected Behavior

dayOfTheWeek() is not in the range { 1 to 7 } as declared.

Actual Behavior

dayOfTheWeek() is in the range { 0 to 6 }

It will not be a problem if the range is declared as { 0 to 6 } in the Library.

SV-Zanshin commented 2 years ago

I've corrected the library so that the range is returned as 1-7 as opposed to 0-6

SV-Zanshin commented 2 years ago

Created a new release of 1.0.9