Wireshark · Wireshark-dev: Re: [Wireshark-dev] review_for_checkin requested: [Bug 1789] packet-c*.c display filter field proble

[Richard van der Hoff <richardv@xxxxxxxxxxxxx>]

bugzilla-request-daemon@xxxxxxxxxxxxx wrote:
> Chris Maynard <christopher.maynard@xxxxxxxxx> has asked  for
> review_for_checkin:
> Bug 1789: packet-c*.c display filter field problems
> http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=1789
>
> ------- Additional Comments from Chris Maynard <christopher.maynard@xxxxxxxxx>
> Is this better or still not quite what you're expecting?

I think I'd rather a single patch file than a zip of several? Mostly 
because then I can look at them in my browser without downloading them 
:). But yup, that's better, thanks!

I think, however, that before you spend too much more time on this, 
somebody ought to review what you've done so far to check that we're all 
on the same page with respect to what are good changes. It would be a 
shame if you spent weeks on it and we had to say "uh, no, you've got the 
wrong idea."

Unfortunately, I'm a bit busy to volunteer...