diff options
author | Ivy Foster <ivy.foster@gmail.com> | 2016-10-12 22:13:31 +0200 |
---|---|---|
committer | Allan McRae <allan@archlinux.org> | 2016-10-22 12:50:55 +0200 |
commit | a55adb81d0f6fcd7fe98cc444806b3b0d25efc9c (patch) | |
tree | 8bf6a8b9f048a9b1073d964ffcead9015f404bd0 /lib/libalpm/be_sync.c | |
parent | a79be86f5751b4341f1339a32a5a8737439ac28d (diff) | |
download | pacman-a55adb81d0f6fcd7fe98cc444806b3b0d25efc9c.tar.gz pacman-a55adb81d0f6fcd7fe98cc444806b3b0d25efc9c.tar.xz |
Add ALPM_ERR_OK to _alpm_errno_t
This allows functions which return an _alpm_errno_t to always return a
genuine _alpm_errno_t for consistency, even in cases where there are
no errors. Since ALPM_ERR_OK = 0, their callers can still simply check
'err = some_fn(); if (!err) { ... }'.
Signed-off-by: Ivy Foster <ivy.foster@gmail.com>
Signed-off-by: Allan McRae <allan@archlinux.org>
Diffstat (limited to 'lib/libalpm/be_sync.c')
-rw-r--r-- | lib/libalpm/be_sync.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/libalpm/be_sync.c b/lib/libalpm/be_sync.c index a8362778..856aa420 100644 --- a/lib/libalpm/be_sync.c +++ b/lib/libalpm/be_sync.c @@ -186,7 +186,7 @@ int SYMEXPORT alpm_db_update(int force, alpm_db_t *db) /* Sanity checks */ ASSERT(db != NULL, return -1); handle = db->handle; - handle->pm_errno = 0; + handle->pm_errno = ALPM_ERR_OK; ASSERT(db != handle->db_local, RET_ERR(handle, ALPM_ERR_WRONG_ARGS, -1)); ASSERT(db->servers != NULL, RET_ERR(handle, ALPM_ERR_SERVER_NONE, -1)); @@ -330,7 +330,7 @@ int SYMEXPORT alpm_db_update(int force, alpm_db_t *db) _alpm_log(handle, ALPM_LOG_DEBUG, "failed to sync db: %s\n", alpm_strerror(handle->pm_errno)); } else { - handle->pm_errno = 0; + handle->pm_errno = ALPM_ERR_OK; } _alpm_handle_unlock(handle); |