diff options
| author | Dan McGee <dan@archlinux.org> | 2011-08-17 23:25:19 -0500 |
|---|---|---|
| committer | Dan McGee <dan@archlinux.org> | 2011-08-18 10:47:41 -0500 |
| commit | c4bd476ad13e142fe8323fe74d84b3950b53da17 (patch) | |
| tree | ddfd2afb08bf94b300eecee7cfef3394ed8c1092 /lib/libalpm/remove.c | |
| parent | c885a953eb888004f0302ed3eceafef93e2f072f (diff) | |
| download | pacman-c4bd476ad13e142fe8323fe74d84b3950b53da17.tar.xz | |
Remove use of no-op accessor functions in library
The functions alpm_db_get_name(), alpm_pkg_get_name(), and
alpm_pkg_get_version() are not necessary at all, so remove the calling
and indirection when used in the backend, which makes things slightly
more efficient and reduces code size.
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'lib/libalpm/remove.c')
| -rw-r--r-- | lib/libalpm/remove.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/libalpm/remove.c b/lib/libalpm/remove.c index 0afa2656..c6886c6f 100644 --- a/lib/libalpm/remove.c +++ b/lib/libalpm/remove.c @@ -81,9 +81,9 @@ static void remove_prepare_cascade(alpm_handle_t *handle, alpm_list_t *lp) alpm_depmissing_t *miss = i->data; alpm_pkg_t *info = _alpm_db_get_pkgfromcache(handle->db_local, miss->target); if(info) { - if(!_alpm_pkg_find(trans->remove, alpm_pkg_get_name(info))) { + if(!_alpm_pkg_find(trans->remove, info->name)) { _alpm_log(handle, ALPM_LOG_DEBUG, "pulling %s in target list\n", - alpm_pkg_get_name(info)); + info->name); trans->remove = alpm_list_add(trans->remove, _alpm_pkg_dup(info)); } } else { @@ -117,7 +117,7 @@ static void remove_prepare_keep_needed(alpm_handle_t *handle, alpm_list_t *lp) pkg = vpkg; if(pkg) { _alpm_log(handle, ALPM_LOG_WARNING, _("removing %s from target list\n"), - alpm_pkg_get_name(pkg)); + pkg->name); _alpm_pkg_free(pkg); } } |