Wireshark-dev: Re: [Wireshark-dev] review_for_checkin requested: [Bug 1789] packet-c*.c display
From: Richard van der Hoff <[email protected]>
Date: Thu, 23 Aug 2007 21:19:13 +0100
[email protected] wrote:
Chris Maynard <[email protected]> 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 <[email protected]>
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...