Re: Patching as per checkAPI

[Bill Meier <wmeier () newsguy com>]

On 6/18/2014 3:40 AM, Evan Huus wrote:
> If they're distinct changes on a single branch (so they're related, but
> make sense as individual commits in the final repo) then #2. Gerrit
> might warn you, but I think it will let you do it as long as you confirm
> you mean it.

One push with multiple commits should work just fine.

Each commit (with its separate Gerrit ChangeId) will create a separate 
patch in Gerrit.

I suggest that changes related to specific APIs (e.g., tvb_length() --> 
tvb_captured_length()) be grouped in commits so as to minimize the 
effort required to review and submit the patches in Gerrit.

Bill


___________________________________________________________________________
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