Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-dev: Re: [Wireshark-dev] Wireshark code review

From: Niels de Vos <ndevos@xxxxxxxxxx>
Date: Wed, 25 Mar 2015 10:03:28 +0100
On Wed, Mar 25, 2015 at 09:15:52AM +0100, Dario Lombardo wrote:
> Since you now have 2 changes submitted, you should abandon one of them (do
> it from the web interface), then follow Alexis' suggestion about sqashing
> and amending, then push the final revision into the survived change (do it
> using the correct change-id, read it from the web interface again).
> 
> I don't know what happens if one push a change-x from a branch b1, then
> push the same change-x from another branch b2. Maybe the latter overwrites
> the first in gerrit? Anyone tried it?

It is a valid use to have the same change-id in different branches. It
makes it possible to easily find backports that way, for example. If a
change was posted to the wrong upstream branch in Gerrit, you will need
to abandon that change and post it to the correct branch.

Posting a patch with a change-id that is already in a different branch,
will not overwrite/abandon the already existing patch.

HTH,
Niels

> 
> On Wed, Mar 25, 2015 at 7:10 AM, Anil <anilkumar911@xxxxxxxxx> wrote:
> 
> > Hi All --
> >
> > I have done a mistake with submitting code for review.
> >
> > My initial review is https://code.wireshark.org/review/#/c/7751
> > <https://code.wireshark.org/review/#/c/7751/1/epan/dissectors/packet-nstrace.c>
> >
> > I fixed the issues reported there and have checked in the new changes. The
> > problem is that the new changes are in a different branch than the original
> > one. So it has created a new review
> > https://code.wireshark.org/review/#/c/7802/
> >
> > I need some help about what can be done now ?
> >
> > --Anil
> >
> > ___________________________________________________________________________
> > Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
> > Archives:    https://www.wireshark.org/lists/wireshark-dev
> > Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
> >              mailto:wireshark-dev-request@xxxxxxxxxxxxx
> > ?subject=unsubscribe
> >

> ___________________________________________________________________________
> Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
> Archives:    https://www.wireshark.org/lists/wireshark-dev
> Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
>              mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe