Wireshark-commits: [Wireshark-commits] master 0162e54: Clean up includes of unistd.h, fcntl.h, and
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sat, 7 Nov 2015 21:52:32 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=0162e54075bb7ad967a85736b3c8addd53d86de5 Submitter: Guy Harris (guy@xxxxxxxxxxxx) Changed: branch: master Repository: wireshark Commits: 0162e54 by Guy Harris (guy@xxxxxxxxxxxx): Clean up includes of unistd.h, fcntl.h, and sys/stat.h. Have wsutil/file_util.h include them on UN*X, just as it includes io.h on Windows, so we can have a rule of "if you do file operations, include <wsutil/file_util.h> and use the routines in it". Remove includes of unistd.h, fcntl.h, and sys/stat.h that aren't necessary (whether because of the addition of them to wsutil/file_util.h or because they weren't needed in the first place). Change-Id: Ie241dd74deff284e39a5f690a297dbb6e1dc485f Reviewed-on: https://code.wireshark.org/review/11619 Petri-Dish: Guy Harris <guy@xxxxxxxxxxxx> Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx> Actions performed: from 24bfb7e column-utils: Refactor col_append_port() to col_append_ports() adds 0162e54 Clean up includes of unistd.h, fcntl.h, and sys/stat.h. Summary of changes: capchild/capture_sync.c | 8 -------- capinfos.c | 4 ---- capture_opts.c | 4 ---- capture_stop_conditions.c | 1 - captype.c | 4 ---- dumpcap.c | 12 ------------ epan/addr_resolv.c | 4 ---- epan/disabled_protos.c | 4 ---- epan/dissectors/packet-lmp.c | 4 ---- epan/dissectors/packet-rsvp.c | 4 ---- epan/prefs.c | 4 ---- file.c | 8 -------- fileset.c | 12 ------------ mergecap.c | 8 -------- plugins/mate/mate_util.h | 4 ---- randpkt.c | 8 -------- rawshark.c | 12 ------------ reordercap.c | 4 ---- ringbuffer.c | 8 -------- text2pcap.c | 4 ---- tfshark.c | 12 ------------ tshark.c | 12 ------------ ui/capture.c | 4 ---- ui/cli/tap-follow.c | 4 ---- ui/export_object.c | 8 -------- ui/filters.c | 4 ---- ui/gtk/capture_file_dlg.c | 4 ---- ui/gtk/export_sslkeys.c | 8 -------- ui/gtk/fileset_dlg.c | 4 ---- ui/gtk/firewall_dlg.c | 8 -------- ui/gtk/follow_stream.c | 3 --- ui/gtk/follow_tcp.c | 4 ---- ui/gtk/iax2_analysis.c | 8 -------- ui/gtk/main.c | 4 ---- ui/gtk/packet_panes.c | 8 -------- ui/gtk/rtp_analysis.c | 8 -------- ui/mcast_stream.c | 3 --- ui/qt/main_window_slots.cpp | 8 -------- ui/rtp_stream.c | 4 ---- ui/text_import.c | 4 ---- ui/traffic_table_ui.c | 3 --- wiretap/ber.c | 4 ---- wiretap/file_access.c | 8 -------- wiretap/file_wrappers.c | 7 ------- wiretap/stanag4607.c | 4 ---- wiretap/tnef.c | 4 ---- wiretap/wtap.c | 4 ---- wsutil/file_util.h | 20 +++++++++++++------- wsutil/filesystem.c | 12 ------------ wsutil/plugins.c | 4 ---- wsutil/tempfile.c | 8 -------- wsutil/u3.c | 6 +----- 52 files changed, 14 insertions(+), 313 deletions(-)
- Prev by Date: [Wireshark-commits] master 24bfb7e: column-utils: Refactor col_append_port() to col_append_ports()
- Next by Date: [Wireshark-commits] master 4897ef1: Just use <wsutil/file_util.h>.
- Previous by thread: [Wireshark-commits] master 24bfb7e: column-utils: Refactor col_append_port() to col_append_ports()
- Next by thread: [Wireshark-commits] master 4897ef1: Just use <wsutil/file_util.h>.
- Index(es):