From 40ea6cd607d5edb2699c084259b237d435313670 Mon Sep 17 00:00:00 2001
From: Dan McGee <dan@archlinux.org>
Date: Mon, 8 Aug 2011 23:57:19 -0500
Subject: Depend on name_hash being set
This is a fairly valid assumption at this point, or at least as good of
one as assuming packages all have names.
Signed-off-by: Dan McGee <dan@archlinux.org>
---
lib/libalpm/deps.c | 3 +--
lib/libalpm/package.c | 3 +--
2 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/lib/libalpm/deps.c b/lib/libalpm/deps.c
index c3681b34..aef66d2d 100644
--- a/lib/libalpm/deps.c
+++ b/lib/libalpm/deps.c
@@ -373,8 +373,7 @@ int _alpm_depcmp(alpm_pkg_t *pkg, alpm_depend_t *dep)
int satisfy = 0;
/* check (pkg->name, pkg->version) */
- if(pkg->name_hash && dep->name_hash
- && pkg->name_hash != dep->name_hash) {
+ if(pkg->name_hash != dep->name_hash) {
/* skip more expensive checks */
} else {
satisfy = (strcmp(pkg->name, dep->name) == 0
diff --git a/lib/libalpm/package.c b/lib/libalpm/package.c
index fd3d0c65..947f62e0 100644
--- a/lib/libalpm/package.c
+++ b/lib/libalpm/package.c
@@ -625,8 +625,7 @@ alpm_pkg_t *_alpm_pkg_find(alpm_list_t *haystack, const char *needle)
alpm_pkg_t *info = lp->data;
if(info) {
- /* a zero hash will cause a fall-through just in case */
- if(info->name_hash && info->name_hash != needle_hash) {
+ if(info->name_hash != needle_hash) {
continue;
}
--
cgit v1.2.3-2-g168b