Wireshark mailing list archives
Re: Clear packet comment should remove the comment entirely?
From: Anders Broman <anders.broman () ericsson com>
Date: Mon, 2 Apr 2012 17:22:08 +0200
-----Original Message----- From: wireshark-dev-bounces () wireshark org [mailto:wireshark-dev-bounces () wireshark org] On Behalf Of Jeff Morriss Sent: den 2 april 2012 16:53 To: Developer support list for Wireshark Subject: Re: [Wireshark-dev] Clear packet comment should remove the comment entirely? Anders Broman wrote:
Stephen Fisher skrev 2012-03-30 17:59:Unfortunately, I haven't been following development much lately. I see that we have a packet comment feature now. Is there a way to remove a comment entirely? I envision the clear button in the "Edit or Add Packet Comments" could do this instead of just clearing the text buffer, but I see that functionality would probably have to go down into the new packet list code and further into the frame data's opt_comment variable.If you press the clear button and then save the file and reload the saved file I think the comment should be gone.I'd been thinking about this some too; one of the first few things I wanted to do in playing with it was to Delete the comment (from the file, from the packet). >I imagine users will want to too; I think leaving an empty comment feels like a workaround (even if it really deletes the comment from the file). I just didn't (and probably won't soon) have time to do anything about it.
Fixed in revision 41895. Regards Anders ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Re: Clear packet comment should remove the comment entirely? Jeff Morriss (Apr 02)
- Re: Clear packet comment should remove the comment entirely? Anders Broman (Apr 02)
- Re: Clear packet comment should remove the comment entirely? Jeff Morriss (Apr 02)
- Re: Clear packet comment should remove the comment entirely? Anders Broman (Apr 02)
- Re: Clear packet comment should remove the comment entirely? Jeff Morriss (Apr 02)
- Re: Clear packet comment should remove the comment entirely? Jeff Morriss (Apr 02)
- Re: Clear packet comment should remove the comment entirely? Anders Broman (Apr 02)