Re: [Wireshark-dev] Top 3 patches to review ;)

2012-02-06 Thread Tony Trinh
My opinion for 5575 is "not [yet]" (clarification and cleanup needed, especially for patch 7448 ). On Sun, Feb 5, 2012 at 10:34 AM, Anders Broman wrote: > Hi, > I'd like to hear t

[Wireshark-dev] Top 3 patches to review ;)

2012-02-05 Thread Anders Broman
Hi, I'd like to hear the opinion on these patches, apply or not? fix in a different way... 3984: r29723 optimization, if (cinfo->col_data[col] != cinfo->col_buf[col]) col_data is not always a constant https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3984 Patch to fix memory leaks/errors i