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/be_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/be_package.c')
| -rw-r--r-- | lib/libalpm/be_package.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/lib/libalpm/be_package.c b/lib/libalpm/be_package.c index 9c7c1617..1cd30c28 100644 --- a/lib/libalpm/be_package.c +++ b/lib/libalpm/be_package.c @@ -132,7 +132,7 @@ static int parse_descfile(struct archive *a, pmpkg_t *newpkg) * through the full archive * @return An information filled pmpkg_t struct */ -static pmpkg_t *pkg_load(const char *pkgfile, unsigned short full) +static pmpkg_t *pkg_load(const char *pkgfile, int full) { int ret = ARCHIVE_OK; int config = 0; @@ -269,8 +269,7 @@ error: * @param pkg address of the package pointer * @return 0 on success, -1 on error (pm_errno is set accordingly) */ -int SYMEXPORT alpm_pkg_load(const char *filename, unsigned short full, - pmpkg_t **pkg) +int SYMEXPORT alpm_pkg_load(const char *filename, int full, pmpkg_t **pkg) { ALPM_LOG_FUNC; |