Wireshark mailing list archives
Re: Wireshark code review
From: Dario Lombardo <dario.lombardo.ml () gmail com>
Date: Wed, 25 Mar 2015 09:15:52 +0100
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? On Wed, Mar 25, 2015 at 7:10 AM, Anil <anilkumar911 () gmail com> 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 () wireshark org> Archives: https://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: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Wireshark code review Anil (Mar 24)
- Re: Wireshark code review Alexis La Goutte (Mar 24)
- Re: Wireshark code review Dario Lombardo (Mar 25)
- Re: Wireshark code review Niels de Vos (Mar 25)
- Re: Wireshark code review Alexis La Goutte (Mar 25)
- Re: Wireshark code review Dario Lombardo (Mar 25)