search

HalosGhost / shaman

A small, native C library and utility to fetch weather
GNU General Public License v2.0
49 stars 4 forks source link

st_mtime fix #25

Closed arsv closed 10 years ago

arsv commented 10 years ago

st_mtim may or may not be visible, stat(2) says it requires _BSD_SOURCE and my gcc actually throws an error there with -std=c11. As long as nanoseconds are not needed, it's better to use common st_mtime.

Also, with -Wall -Wpedantic gcc complains about unused variables.

HalosGhost commented 10 years ago

These both seem fine to me, and preliminarily, it looks like it still passes the test suite with flying colors.