Re: [Wireshark-dev] removing mergecap -T option

2015-08-14 Thread Guy Harris
On Aug 14, 2015, at 9:18 AM, Hadriel Kaplan wrote: > Due to some reported bugs and inconsistencies, I'm refactoring the > capture file merging code in mergecap.c and file.c's cf_merge_files() > - basically gutting them and putting most of the logic into a common > "merge_files()" function in mer

Re: [Wireshark-dev] removing mergecap -T option

2015-08-14 Thread Pascal Quantin
Le 14 août 2015 6:30 PM, "Hadriel Kaplan" a écrit : > > But isn't that the purpose of editcap? I'm only talking about > removing it from mergecap. Mmm, sorry I did not realize that. No specific objection on my side then. Pascal. > > -hadriel > > On Fri, Aug 14, 2015 at 12:26 PM, Pascal Quantin

Re: [Wireshark-dev] removing mergecap -T option

2015-08-14 Thread Hadriel Kaplan
But isn't that the purpose of editcap? I'm only talking about removing it from mergecap. -hadriel On Fri, Aug 14, 2015 at 12:26 PM, Pascal Quantin wrote: > > Le 14 août 2015 6:18 PM, "Hadriel Kaplan" a > écrit : >> >> Howdy, >> Due to some reported bugs and inconsistencies, I'm refactoring the

Re: [Wireshark-dev] removing mergecap -T option

2015-08-14 Thread Pascal Quantin
Le 14 août 2015 6:18 PM, "Hadriel Kaplan" a écrit : > > Howdy, > Due to some reported bugs and inconsistencies, I'm refactoring the > capture file merging code in mergecap.c and file.c's cf_merge_files() > - basically gutting them and putting most of the logic into a common > "merge_files()" funct

[Wireshark-dev] removing mergecap -T option

2015-08-14 Thread Hadriel Kaplan
Howdy, Due to some reported bugs and inconsistencies, I'm refactoring the capture file merging code in mergecap.c and file.c's cf_merge_files() - basically gutting them and putting most of the logic into a common "merge_files()" function in merge.c. While merging their function code logic, I notic