Wireshark mailing list archives

Re: [Wireshark-commits] rev31832: /trunk/epan/dissectors/ /trunk/epan/dissectors/:packet-sflow.c

From: "Anders Broman" <a.broman () telia com>
Date: Wed, 10 Feb 2010 23:37:51 +0100



-----Ursprungligt meddelande-----
Från: wireshark-dev-bounces () wireshark org
[mailto:wireshark-dev-bounces () wireshark org] För Joerg Mayer
Skickat: den 10 februari 2010 22:07
Till: Developer support list for Wireshark
Ämne: Re: [Wireshark-dev] [Wireshark-commits] rev31832:
/trunk/epan/dissectors/ /trunk/epan/dissectors/:packet-sflow.c

On Wed, Feb 10, 2010 at 09:46:20PM +0100, Anders Broman wrote:

You were asked to comment on it 4 moth ago ;)


Yep, and I was aware that properly commenting on it would take a bigger
amount
of time than I was willing to spend on it.

I don't have a problem with you reverting it.


Oh no! That would just put responsibility back on me to eventually take
that
time ;-)

ciao
    Joerg

Well stating your doubts and position didn't take that long did it :-)
Still do you propose to revert the checkin I don't mind either way...
I thought the current version had merit over the old one, if not it should
be reverted.
Regards
Anders
-- 
Joerg Mayer                                           <jmayer () loplof de>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.
___________________________________________________________________________
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: [Wireshark-commits] rev 31832: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-sflow.c Joerg Mayer (Feb 10)
- Re: [Wireshark-commits] rev 31832: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-sflow.c Anders Broman (Feb 10)
  - Re: [Wireshark-commits] rev 31832: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-sflow.c Joerg Mayer (Feb 10)
    - Re: [Wireshark-commits] rev31832: /trunk/epan/dissectors/ /trunk/epan/dissectors/:packet-sflow.c Anders Broman (Feb 10)
    - Re: [Wireshark-commits] rev31832: /trunk/epan/dissectors/ /trunk/epan/dissectors/:packet-sflow.c Joerg Mayer (Feb 14)