Commit de2220a48fcf7901432f8093d3223e65f4072ff7

Shawn O. Pearce 2008-10-31T18:16:26

Replace git_result_t with int This seems to be preferred on the mailing list. Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

diff --git a/CONVENTIONS b/CONVENTIONS
index 60811b0..4b09c08 100644
--- a/CONVENTIONS
+++ b/CONVENTIONS
@@ -37,12 +37,12 @@ and may disappear or change their signature in a future release.
 Calling Conventions
 -------------------
 
-Functions should prefer to return a 'git_result_t' to indicate
-success/failure and supply any output through the first argument
-(or first few arguments if multiple outputs are supplied).
+Functions should prefer to return a 'int' to indicate success or
+failure and supply any output through the first argument (or first
+few arguments if multiple outputs are supplied).
 
-git_result_t status codes are 0 for GIT_SUCCESS and < 0 for an
-error.  This permits common POSIX result testing:
+int status codes are 0 for GIT_SUCCESS and < 0 for an error.
+This permits common POSIX result testing:
 
 ----
 	if (git_odb_open(&odb, path))
diff --git a/src/git_common.h b/src/git_common.h
index 6d8e93b..791826c 100644
--- a/src/git_common.h
+++ b/src/git_common.h
@@ -62,9 +62,6 @@ GIT_BEGIN_DECL
 # define GIT_EXTERN(type) type
 #endif
 
-/** Generic result code for any API call. */
-typedef int git_result_t;
-
 /** Operation completed successfully. */
 #define GIT_SUCCESS 0
 
diff --git a/src/git_odb.c b/src/git_odb.c
index 13de54b..d7ae061 100644
--- a/src/git_odb.c
+++ b/src/git_odb.c
@@ -46,7 +46,7 @@ struct git_odb_t {
 	unsigned n_alternates;
 };
 
-git_result_t git_odb_read(
+int git_odb_read(
 	git_sobj_t *out,
 	git_odb_t *db,
 	const git_oid_t *id)
diff --git a/src/git_odb.h b/src/git_odb.h
index 1213764..8e0a161 100644
--- a/src/git_odb.h
+++ b/src/git_odb.h
@@ -60,7 +60,7 @@ typedef struct git_odb_t git_odb_t;
  * @return GIT_SUCCESS if the database opened; otherwise an error
  *         code describing why the open was not possible.
  */
-GIT_EXTERN(git_result_t) git_odb_open(git_odb_t **out, const char *objects_dir);
+GIT_EXTERN(int) git_odb_open(git_odb_t **out, const char *objects_dir);
 
 /**
  * Close an open object database.
@@ -101,7 +101,7 @@ typedef struct {
  * - GIT_SUCCESS if the object was read;
  * - GIT_ENOTFOUND if the object is not in the database.
  */
-GIT_EXTERN(git_result_t) git_odb_read(git_sobj_t *out, git_odb_t *db, const git_oid_t *id);
+GIT_EXTERN(int) git_odb_read(git_sobj_t *out, git_odb_t *db, const git_oid_t *id);
 
 /**
  * Read a small object from the database using only pack files.
@@ -115,7 +115,7 @@ GIT_EXTERN(git_result_t) git_odb_read(git_sobj_t *out, git_odb_t *db, const git_
  * - GIT_SUCCESS if the object was read.
  * - GIT_ENOTFOUND if the object is not in the database.
  */
-GIT_EXTERN(git_result_t) git_odb__read_packed(git_sobj_t *out, git_odb_t *db, const git_oid_t *id);
+GIT_EXTERN(int) git_odb__read_packed(git_sobj_t *out, git_odb_t *db, const git_oid_t *id);
 
 /**
  * Read a small object from the database using only loose object files.
@@ -129,7 +129,7 @@ GIT_EXTERN(git_result_t) git_odb__read_packed(git_sobj_t *out, git_odb_t *db, co
  * - GIT_SUCCESS if the object was read.
  * - GIT_ENOTFOUND if the object is not in the database.
  */
-GIT_EXTERN(git_result_t) git_odb__read_loose(git_sobj_t *out, git_odb_t *db, const git_oid_t *id);
+GIT_EXTERN(int) git_odb__read_loose(git_sobj_t *out, git_odb_t *db, const git_oid_t *id);
 
 /**
  * Release all memory used by the sobj structure.
diff --git a/src/git_oid.c b/src/git_oid.c
index fd899f4..b20f1d8 100644
--- a/src/git_oid.c
+++ b/src/git_oid.c
@@ -55,7 +55,7 @@ static signed char from_hex[] = {
 -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, /* f0 */
 };
 
-git_result_t git_oid_mkstr(git_oid_t *out, const char *str)
+int git_oid_mkstr(git_oid_t *out, const char *str)
 {
 	int p;
 	for (p = 0; p < sizeof(out->id); p++, str += 2) {
diff --git a/src/git_oid.h b/src/git_oid.h
index 30c62a2..9a9e5ab 100644
--- a/src/git_oid.h
+++ b/src/git_oid.h
@@ -62,7 +62,7 @@ typedef struct
  *        needed for an oid encoded in hex (40 bytes).
  * @return GIT_SUCCESS if valid; GIT_ENOTOID on failure.
  */
-GIT_EXTERN(git_result_t) git_oid_mkstr(git_oid_t *out, const char *str);
+GIT_EXTERN(int) git_oid_mkstr(git_oid_t *out, const char *str);
 
 /**
  * Copy an already raw oid into a git_oid structure.
diff --git a/src/git_revwalk.c b/src/git_revwalk.c
index 7b7d17f..5e9e1f2 100644
--- a/src/git_revwalk.c
+++ b/src/git_revwalk.c
@@ -39,7 +39,7 @@
 
 struct git_revp_attr_t {
 	size_t app_size;
-	git_result_t (*app_init)(git_commit_t *, void *);
+	int (*app_init)(git_commit_t *, void *);
 };
 
 struct git_revp_t {
diff --git a/src/git_revwalk.h b/src/git_revwalk.h
index a84d130..e7ed799 100644
--- a/src/git_revwalk.h
+++ b/src/git_revwalk.h
@@ -90,7 +90,7 @@ GIT_EXTERN(git_revp_attr_t*) git_revp_attr_alloc(void);
 GIT_EXTERN(void) git_revp_attr_appdata(
 	git_revp_attr_t *attr,
 	size_t size,
-	git_result_t (*init)(git_commit_t *, void *));
+	int (*init)(git_commit_t *, void *));
 
 /**
  * Free a pool configuration.