Thanks Guy. Unfortunately I'm running my script to generate these changes on Windows, which is too forgiving. Will put stdlib.h and math.h on the do-not-remove list for now.
Martin On Mon, Dec 22, 2014 at 9:24 PM, Wireshark code review < code-review-do-not-re...@wireshark.org> wrote: > URL: > https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=d541774a61c25eabe36d77319e66bd2fb0e0133e > Submitter: Guy Harris (g...@alum.mit.edu) > Changed: branch: master > Repository: wireshark > > Commits: > > d541774 by Guy Harris (g...@alum.mit.edu): > > Include <math.h> or <stdlib.h> as necessary to declare functions. > > Various floating-point math functions require <math.h>, and abs() > requires <stdlib.h>. > > Change-Id: Iadba9e0d7168bba6e67d9221e757a85960507742 > Reviewed-on: https://code.wireshark.org/review/5999 > Reviewed-by: Guy Harris <g...@alum.mit.edu> > > > Actions performed: > > from 596669f First batch of unincludes. Last attempt... > adds d541774 Include <math.h> or <stdlib.h> as necessary to declare > functions. > > > Summary of changes: > epan/dissectors/packet-ansi_801.c | 2 +- > epan/dissectors/packet-aprs.c | 1 + > epan/dissectors/packet-dmp.c | 1 + > epan/dissectors/packet-edonkey.c | 1 + > 4 files changed, 4 insertions(+), 1 deletion(-) > ___________________________________________________________________________ > Sent via: Wireshark-commits mailing list < > wireshark-comm...@wireshark.org> > Archives: http://www.wireshark.org/lists/wireshark-commits > Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits > mailto:wireshark-commits-requ...@wireshark.org > ?subject=unsubscribe >
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev@wireshark.org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe