lib/timeutil: support const correctness for pointer parameter

pabigot · nashif · commit cc1594a59a67 · 2019-09-19T20:49:51.000-04:00
timeutil_timegm() does not modify the passed structure, so it should
indicate that in the signature (even though the GNU extension does not).

Signed-off-by: Peter A. Bigot &lt;pab@pabigot.com&gt;
diff --git a/include/sys/timeutil.h b/include/sys/timeutil.h
@@ -32,7 +32,7 @@ extern "C" {
  *
  * @see http://man7.org/linux/man-pages/man3/timegm.3.html
  */
-time_t timeutil_timegm(struct tm *tm);
+time_t timeutil_timegm(const struct tm *tm);
 
 #ifdef __cplusplus
 }
diff --git a/lib/os/timeutil.c b/lib/os/timeutil.c
@@ -48,7 +48,7 @@ static s64_t time_days_from_civil(s64_t y,
  * @return the signed number of seconds between 1970-01-01T00:00:00
  * and the specified time ignoring leap seconds and DST offsets.
  */
-time_t timeutil_timegm(struct tm *tm)
+time_t timeutil_timegm(const struct tm *tm)
 {
 	s64_t y = 1900 + (s64_t)tm->tm_year;
 	unsigned int m = tm->tm_mon + 1;

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ extern "C" {`
`32`	`32`	`*`
`33`	`33`	`* @see http://man7.org/linux/man-pages/man3/timegm.3.html`
`34`	`34`	`*/`
`35`		`-time_t timeutil_timegm(struct tm *tm);`
	`35`	`+time_t timeutil_timegm(const struct tm *tm);`
`36`	`36`
`37`	`37`	`#ifdef __cplusplus`
`38`	`38`	`}`
Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ static s64_t time_days_from_civil(s64_t y,`
`48`	`48`	`* @return the signed number of seconds between 1970-01-01T00:00:00`
`49`	`49`	`* and the specified time ignoring leap seconds and DST offsets.`
`50`	`50`	`*/`
`51`		`-time_t timeutil_timegm(struct tm *tm)`
	`51`	`+time_t timeutil_timegm(const struct tm *tm)`
`52`	`52`	`{`
`53`	`53`	`s64_t y = 1900 + (s64_t)tm->tm_year;`
`54`	`54`	`unsigned int m = tm->tm_mon + 1;`