diff options
| author | Dan McGee <dan@archlinux.org> | 2009-10-11 20:51:47 +0200 |
|---|---|---|
| committer | Dan McGee <dan@archlinux.org> | 2009-10-11 20:51:47 +0200 |
| commit | 35dc9b03149dc2f42022fe5fea80a3aed3207489 (patch) | |
| tree | dbcfa549cf1b8bf30e4d94bad301e6f1f9c12179 /lib/libalpm/package.c | |
| parent | cf0d619670a4d0c67ad67e22bacbb69c203714f5 (diff) | |
| download | pacman-35dc9b03149dc2f42022fe5fea80a3aed3207489.tar.gz pacman-35dc9b03149dc2f42022fe5fea80a3aed3207489.tar.xz | |
int typing: s/unsigned short/int/ in libalpm
After our recent screwup with size_t and ssize_t in the download code, I
found the `-Wsign-conversion` flag to GCC to see if we were doing anything
else boneheaded. I didn't find anything quite as bad, but we did have some
goofups- most of our public unsigned methods would return -1 on error, which
is a bit odd in an unsigned context.
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'lib/libalpm/package.c')
| -rw-r--r-- | lib/libalpm/package.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/libalpm/package.c b/lib/libalpm/package.c index de171661..572b863e 100644 --- a/lib/libalpm/package.c +++ b/lib/libalpm/package.c @@ -295,7 +295,7 @@ alpm_list_t SYMEXPORT *alpm_pkg_get_groups(pmpkg_t *pkg) return pkg->groups; } -unsigned short SYMEXPORT alpm_pkg_has_force(pmpkg_t *pkg) +int SYMEXPORT alpm_pkg_has_force(pmpkg_t *pkg) { ALPM_LOG_FUNC; @@ -541,7 +541,7 @@ int SYMEXPORT alpm_pkg_changelog_close(const pmpkg_t *pkg, void *fp) return(ret); } -unsigned short SYMEXPORT alpm_pkg_has_scriptlet(pmpkg_t *pkg) +int SYMEXPORT alpm_pkg_has_scriptlet(pmpkg_t *pkg) { ALPM_LOG_FUNC; |