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/handle.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/handle.c')
| -rw-r--r-- | lib/libalpm/handle.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/libalpm/handle.c b/lib/libalpm/handle.c index 012d4121..a503baea 100644 --- a/lib/libalpm/handle.c +++ b/lib/libalpm/handle.c @@ -169,7 +169,7 @@ const char SYMEXPORT *alpm_option_get_lockfile() return handle->lockfile; } -unsigned short SYMEXPORT alpm_option_get_usesyslog() +int SYMEXPORT alpm_option_get_usesyslog() { if (handle == NULL) { pm_errno = PM_ERR_HANDLE_NULL; @@ -437,7 +437,7 @@ int SYMEXPORT alpm_option_set_logfile(const char *logfile) return(0); } -void SYMEXPORT alpm_option_set_usesyslog(unsigned short usesyslog) +void SYMEXPORT alpm_option_set_usesyslog(int usesyslog) { handle->usesyslog = usesyslog; } @@ -536,7 +536,7 @@ void SYMEXPORT alpm_option_set_arch(const char *arch) if(arch) handle->arch = strdup(arch); } -void SYMEXPORT alpm_option_set_usedelta(unsigned short usedelta) +void SYMEXPORT alpm_option_set_usedelta(int usedelta) { handle->usedelta = usedelta; } |