rebase: correct the return values for CurrentOperationIndex #396

Merged
carlosmn merged 1 commits from cmn/rebase-no-operation into master 2017-07-08 15:22:55 -05:00
carlosmn commented 2017-07-08 14:36:38 -05:00 (Migrated from github.com)

We were incorectly reporting C.GIT_REBASE_NO_OPERATION as an error code when
it is none. We should instead return it as the value. The compiler doesn't seem
to actually look at the sizes so instead we must recreate the value ourselves
with ^uint(0).

The error return is kept for API compatibility but should go away eventually.

This supersedes #385

We were incorectly reporting `C.GIT_REBASE_NO_OPERATION` as an error code when it is none. We should instead return it as the value. The compiler doesn't seem to actually look at the sizes so instead we must recreate the value ourselves with `^uint(0)`. The error return is kept for API compatibility but should go away eventually. This supersedes #385
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: jcarr/git2go#396
No description provided.