Though I would note that if there are large swaths of an API that haven't been converted yet, that's an indicator that it's not as simple as a search-and-replace, so please don't just submit large changes like that, since it will likely break things in a subtle way (since the new/old APIs behave slightly differently so conversion needs to be done carefully).
Evan On Wed, Jun 18, 2014 at 12:35 AM, Dario Lombardo < [email protected]> wrote: > I'm trying to taking care of some of the warnings from checkAPI. The > result is a set of commits. > What should I do now? > > 1) submit one push for every commit (eg. branch checkapi1, checkapi2, etc.) > > 2) submit one push with multiple commits (don't know if gerrit allows > it... I got an error when I tried to do so) > > 3) squash them into one commit (don't like it... the commits are > related to different APIs) > > 4) something else :) > > Thanks for your help. > Dario. > ___________________________________________________________________________ > Sent via: Wireshark-dev mailing list <[email protected]> > Archives: http://www.wireshark.org/lists/wireshark-dev > Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev > mailto:[email protected] > ?subject=unsubscribe >
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <[email protected]> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:[email protected]?subject=unsubscribe
