add minmea_getdate() - Githubissues

kosma / minmea

a lightweight GPS NMEA 0183 parser library in pure C

Do What The F*ck You Want To Public License

735 stars 246 forks source link

add minmea_getdate() #45

Closed benpicco closed 2 years ago

benpicco commented 5 years ago

I needed a struct tm instead of a Unix timestamp. The struct is already being assembled in minmea_gettime() and converting struct tm -> time_t -> struct tm is silly, so I added a function to make that functionality available for external users.

kosma commented 2 years ago

Needs tests before we can merge.

kosma commented 2 years ago

Merged. Thanks for this contribution, it does indeed make sense and should have been in the library from the beginning!

Changes I did to the original PR:

added tests
renamed the function minmea_getdatetime() as "getdate" is misleading - it doesn't just get the date but date+time
some minor style changes