ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
July 17th, 2024 | 10:00am-11:55am SGT (UTC+8) | Online

Wireshark-bugs: [Wireshark-bugs] [Bug 9047] EPL: Dissection for WriteMultipleParameter, Node Ali

Date: Mon, 09 Sep 2013 20:31:01 +0000

Comment # 13 on bug 9047 from
(In reply to comment #12)
> (In reply to comment #11)
> > Created attachment 11530 [details]
> > For the whitespace changes, I may have taken Michaels comment a little bit
> > to serious and went overboard with them. I did use BeyondCompare to review
> > the changes, and they look good to me.
> My understanding is that Michael suggested you to remove the extra spaces
> added at the end of some lines in your first patch, not to change all the
> indentation of the file.
> I'm still -1 for tis change due to the reasons I gave in comment #10. If
> someone else does not care that much, I will let him commit the patch as-is
> :)

Ok, I'll rework it, and will split the patch up in two parts. The first one
being the proposed changes, the second one will be the indentation changes.


You are receiving this mail because:
  • You are watching all bug changes.