From a267084c257281d28b35319f6b56b75f2777a493 Mon Sep 17 00:00:00 2001 From: Paul Eggert Date: Mon, 27 Aug 2012 23:11:45 -0700 Subject: [PATCH] utimens-tests: avoid unnecessary 'inline' * modules/fdutimensat-tests (configure.ac): * modules/futimens-tests (configure.ac): * modules/utimens-tests (configure.ac): * modules/utimensat-tests (configure.ac): Remove AC_C_INLINE. * tests/test-utimens-common.h (ctime_compare): No longer inline. Simplicity and portability trump efficiency here. --- ChangeLog | 9 +++++++++ modules/fdutimensat-tests | 1 - modules/futimens-tests | 1 - modules/utimens-tests | 1 - modules/utimensat-tests | 1 - tests/test-utimens-common.h | 2 +- 6 files changed, 10 insertions(+), 5 deletions(-) diff --git a/ChangeLog b/ChangeLog index 454ee62f08..79a165a55b 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,14 @@ 2012-09-22 Paul Eggert + utimens-tests: avoid unnecessary 'inline' + * modules/fdutimensat-tests (configure.ac): + * modules/futimens-tests (configure.ac): + * modules/utimens-tests (configure.ac): + * modules/utimensat-tests (configure.ac): + Remove AC_C_INLINE. + * tests/test-utimens-common.h (ctime_compare): + No longer inline. Simplicity and portability trump efficiency here. + misc: don't limit commentary to inline functions * lib/binary-io.h, lib/malloca.h, lib/safe-alloc.c: * lib/xalloc-oversized.h, lib/xsize.h: diff --git a/modules/fdutimensat-tests b/modules/fdutimensat-tests index c4dc5af604..9e3e77979a 100644 --- a/modules/fdutimensat-tests +++ b/modules/fdutimensat-tests @@ -17,7 +17,6 @@ usleep utimecmp configure.ac: -AC_REQUIRE([AC_C_INLINE]) Makefile.am: TESTS += test-fdutimensat diff --git a/modules/futimens-tests b/modules/futimens-tests index 1b72de5130..ff9801961e 100644 --- a/modules/futimens-tests +++ b/modules/futimens-tests @@ -15,7 +15,6 @@ usleep utimecmp configure.ac: -AC_REQUIRE([AC_C_INLINE]) Makefile.am: TESTS += test-futimens diff --git a/modules/utimens-tests b/modules/utimens-tests index 802d8e51d1..4d5d1f576a 100644 --- a/modules/utimens-tests +++ b/modules/utimens-tests @@ -16,7 +16,6 @@ usleep utimecmp configure.ac: -AC_REQUIRE([AC_C_INLINE]) Makefile.am: TESTS += test-utimens diff --git a/modules/utimensat-tests b/modules/utimensat-tests index ec426bbcd6..6acfe89ad6 100644 --- a/modules/utimensat-tests +++ b/modules/utimensat-tests @@ -14,7 +14,6 @@ usleep utimecmp configure.ac: -AC_REQUIRE([AC_C_INLINE]) Makefile.am: TESTS += test-utimensat diff --git a/tests/test-utimens-common.h b/tests/test-utimens-common.h index d1fb0f4121..6fe8465b71 100644 --- a/tests/test-utimens-common.h +++ b/tests/test-utimens-common.h @@ -59,7 +59,7 @@ enum { /* Compare two st_ctime values. Return -1, 0 or 1, respectively when A's st_ctime is smaller than, equal to or greater than B's. */ -static inline int +static int ctime_compare (struct stat const *a, struct stat const *b) { if (a->st_ctime < b->st_ctime) -- 2.39.5