Wireshark · Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] master d541774: Include <math.h> or <stdlib.h> as necessary

[Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx>]

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-reply@xxxxxxxxxxxxx> wrote:

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=d541774a61c25eabe36d77319e66bd2fb0e0133e
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

d541774 by Guy Harris (guy@xxxxxxxxxxxx):

    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 <guy@xxxxxxxxxxxx>


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-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