ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
April 17th, 2024 | 14:30-16:00 SGT (UTC+8) | Online

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

From: Richard van der Hoff <richardv@xxxxxxxxxxxxx>
Date: Thu, 23 Aug 2007 21:19:13 +0100
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...