rebase: updates based on PR comments

include/git2.h

@@ -40,6 +40,7 @@
 #include "git2/patch.h"
 #include "git2/pathspec.h"
 #include "git2/push.h"
+#include "git2/rebase.h"
 #include "git2/refdb.h"
 #include "git2/reflog.h"
 #include "git2/refs.h"

include/git2/rebase.h

@@ -84,9 +84,8 @@ typedef enum {
 
 typedef struct {
 	/** The type of rebase operation. */
-	unsigned int type;
+	git_rebase_operation_t type;
 
-	union {
 	/**
 	 * The commit ID being cherry-picked.  This will be populated for
 	 * all operations except those of type `GIT_REBASE_OPERATION_EXEC`.
@@ -98,7 +97,6 @@ typedef struct {
 	 * be populated for operations of type `GIT_REBASE_OPERATION_EXEC`.
 	 */
 	const char *exec;
-	};
 } git_rebase_operation;
 
 /**
@@ -115,7 +113,7 @@ GIT_EXTERN(int) git_rebase_init_options(
 	unsigned int version);
 
 /**
- * Initializes a rebase operation to rebase the changes in `ours`
+ * Initializes a rebase operation to rebase the changes in `branch`
  * relative to `upstream` onto another branch.  To begin the rebase
  * process, call `git_rebase_next`.  When you have finished with this
  * object, call `git_rebase_free`.
@@ -127,7 +125,7 @@ GIT_EXTERN(int) git_rebase_init_options(
  *                 reachable commits
  * @param onto The branch to rebase onto, or NULL to rebase onto the given
  *             upstream
- * @param signature The signature of the rebaser
+ * @param signature The signature of the rebaser (optional)
  * @param opts Options to specify how rebase is performed
  * @return Zero on success; -1 on failure.
  */
@@ -242,7 +240,7 @@ GIT_EXTERN(int) git_rebase_abort(
  * been applied.
  *
  * @param rebase The rebase that is in-progress
- * @param signature The identity that is finishing the rebase
+ * @param signature The identity that is finishing the rebase (optional)
  * @param opts Options to specify how rebase is finished
  * @param Zero on success; -1 on error
  */
@@ -254,7 +252,7 @@ GIT_EXTERN(int) git_rebase_finish(
 /**
  * Frees the `git_rebase` object.
  *
- * @param rebase The rebase that is in-progress
+ * @param rebase The rebase object
  */
 GIT_EXTERN(void) git_rebase_free(git_rebase *rebase);
 

src/rebase.c

@@ -948,6 +948,7 @@ static int rebase_copy_note(
 {
 	git_note *note = NULL;
 	git_oid note_id;
+	git_signature *who = NULL;
 	int error;
 
 	if ((error = git_note_read(&note, rebase->repo, opts->rewrite_notes_ref, from)) < 0) {
@@ -959,11 +960,24 @@ static int rebase_copy_note(
 		goto done;
 	}
 
+	if (!committer) {
+		if((error = git_signature_default(&who, rebase->repo)) < 0) {
+			if (error != GIT_ENOTFOUND ||
+				(error = git_signature_now(&who, "unknown", "unknown")) < 0)
+				goto done;
+
+			giterr_clear();
+		}
+
+		committer = who;
+	}
+
 	error = git_note_create(&note_id, rebase->repo, git_note_author(note),
 		committer, opts->rewrite_notes_ref, to, git_note_message(note), 0);
 
 done:
 	git_note_free(note);
+	git_signature_free(who);
 
 	return error;
 }