Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] master b736896: Handle floorl() the same

From: Stephen Fisher <sfisher@xxxxxxx>
Date: Fri, 19 Dec 2014 20:08:01 -0700
Thanks for correcting this.  I see now that there are other functions 
defined the way you re-defined floorl().  The example I randomly picked 
to follow was wsutil/tempfile.[ch], which is done with just 
AC_CHECK_FUNCS() and #ifndef HAVE_MKSTEMP.  Does that need to be 
corrected too then?

On Sat, Dec 20, 2014 at 02:54:08AM +0000, Wireshark code review wrote:
> URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=b73689649059060c5876a8ae281a2ded58a1b8fc
> Submitter: Guy Harris (guy@xxxxxxxxxxxx)
> Changed: branch: master
> Repository: wireshark
> 
> Commits:
> 
> b736896 by Guy Harris (guy@xxxxxxxxxxxx):
> 
>     Handle floorl() the same way we handle other not-on-all-platforms functions.
>     
>     Use AC_CHECK_FUNC() for it, define FLOORL_LO to floorl.lo if we *don't*
>     have it, add FLOORL_LO to the list of items conditionally built in
>     libwsutil, and include "wsutil/floor.h" only if HAVE_FLOORL is *not*
>     defined, as that means it's *not* supplied by the platform and thus
>     *not* declared in <math.h>.
>     
>     Also, use the standard export stuff in wsutil/floor.h.
>     
>     Change-Id: Ic24aa69f65f2d15450d8b84b0c2b0c58f38edebe
>     Reviewed-on: https://code.wireshark.org/review/5901
>     Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
>     
> 
> Actions performed:
> 
>     from  201a7e7   Change more variables from si_code to mtp3_si_code to avoid conflicts with a #define in a NetBSD header file.
>     adds  b736896   Handle floorl() the same way we handle other not-on-all-platforms functions.
> 
> 
> Summary of changes:
>  configure.ac       |    9 ++++++++-
>  ui/time_shift.c    |    4 +++-
>  wsutil/Makefile.am |    1 +
>  wsutil/floor.h     |   14 ++++++++++----
>  4 files changed, 22 insertions(+), 6 deletions(-)
> ___________________________________________________________________________
> Sent via:    Wireshark-commits mailing list <wireshark-commits@xxxxxxxxxxxxx>
> Archives:    http://www.wireshark.org/lists/wireshark-commits
> Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
>              mailto:wireshark-commits-request@xxxxxxxxxxxxx?subject=unsubscribe