Wireshark · Ethereal-dev: Re: [ethereal-dev] Patch to "packet-arp.c" checked in

[Laurent Deniel <deniel@xxxxxxxxxxx>]

Guy Harris wrote:
> 
> That's one of the problems with multi-developer projects - you run the
> risk of having to merge two developer's changes to the same file (and
> given the amount of stuff I want from a network sniffing program, and
> stuff I have lying around in another such program on which I'd been
> working, there's probably a significant risk there...).
>
 [...]
> Another approach is to have changes by people other than the "owner" of
> a module be submitted as patches for somebody to merge; that leaves that
> somebody, whether it be the owner or, perhaps, Gerald, to do the
> merging (and, on some occasions, cleaning up the result).
> 

I agree. But for the FDDI stuff it was a little different since it was a 
first preliminary version (even not checked in the CVS tree for your first 
patch, and never delivered in tar version), so you should expect more work 
of the original author and send patch directly to him before checking in 
your changes. But forget it, they are in the right direction.

Regards,

Laurent.

--
Laurent DENIEL            | E-mail: deniel@xxxxxxxxxxx
Paris, FRANCE             |         deniel@xxxxxxxxxxxxxxxxxxxxxxxxxxxx
                          | WWW   : http://www.worldnet.fr/~deniel
    All above opinions are personal, unless stated otherwise.