[pacman-dev] [PATCH 2/2] More consistent printing of off_t and time_t

Xavier Chantry shiningxc at gmail.com
Sat Nov 14 14:30:44 EST 2009


%ld for type_t
%"PRId64" for off_t

Signed-off-by: Xavier Chantry <shiningxc at gmail.com>
---
 lib/libalpm/add.c      |    7 +++----
 lib/libalpm/be_files.c |   12 ++++++------
 lib/libalpm/delta.c    |    6 +++---
 lib/libalpm/dload.c    |    4 ++--
 lib/libalpm/sync.c     |    6 +++---
 5 files changed, 17 insertions(+), 18 deletions(-)

diff --git a/lib/libalpm/add.c b/lib/libalpm/add.c
index ebcd6a5..05bb8d4 100644
--- a/lib/libalpm/add.c
+++ b/lib/libalpm/add.c
@@ -28,8 +28,7 @@
 #include <sys/types.h>
 #include <sys/stat.h>
 #include <unistd.h>
-#include <inttypes.h> /* int64_t */
-#include <stdint.h> /* intmax_t */
+#include <inttypes.h> /* int64_t,PRId64 */
 
 /* libarchive */
 #include <archive.h>
@@ -602,8 +601,8 @@ static int commit_single_pkg(pmpkg_t *newpkg, int pkg_current, int pkg_count,
 				int64_t pos = archive_position_compressed(archive);
 				percent = (double)pos / (double)newpkg->size;
 				_alpm_log(PM_LOG_DEBUG, "decompression progress: "
-						"%f%% (%"PRId64" / %jd)\n",
-						percent*100.0, pos, (intmax_t)newpkg->size);
+						"%f%% (%"PRId64" / %"PRId64")\n",
+						percent*100.0, pos, newpkg->size);
 				if(percent >= 1.0) {
 					percent = 1.0;
 				}
diff --git a/lib/libalpm/be_files.c b/lib/libalpm/be_files.c
index 90e97a5..dc95bdb 100644
--- a/lib/libalpm/be_files.c
+++ b/lib/libalpm/be_files.c
@@ -25,7 +25,7 @@
 #include <stdlib.h>
 #include <errno.h>
 #include <string.h>
-#include <stdint.h> /* uintmax_t, intmax_t */
+#include <inttypes.h> /* PRId64 */
 #include <sys/stat.h>
 #include <dirent.h>
 #include <ctype.h>
@@ -697,11 +697,11 @@ int _alpm_db_write(pmdb_t *db, pmpkg_t *info, pmdbinfrq_t inforeq)
 			}
 			if(info->builddate) {
 				fprintf(fp, "%%BUILDDATE%%\n"
-								"%ju\n\n", (uintmax_t)info->builddate);
+								"%ld\n\n", info->builddate);
 			}
 			if(info->installdate) {
 				fprintf(fp, "%%INSTALLDATE%%\n"
-								"%ju\n\n", (uintmax_t)info->installdate);
+								"%ld\n\n", info->installdate);
 			}
 			if(info->packager) {
 				fprintf(fp, "%%PACKAGER%%\n"
@@ -710,7 +710,7 @@ int _alpm_db_write(pmdb_t *db, pmpkg_t *info, pmdbinfrq_t inforeq)
 			if(info->isize) {
 				/* only write installed size, csize is irrelevant once installed */
 				fprintf(fp, "%%SIZE%%\n"
-								"%jd\n\n", (intmax_t)info->isize);
+								"%"PRId64"\n\n", info->isize);
 			}
 			if(info->reason) {
 				fprintf(fp, "%%REASON%%\n"
@@ -719,11 +719,11 @@ int _alpm_db_write(pmdb_t *db, pmpkg_t *info, pmdbinfrq_t inforeq)
 		} else {
 			if(info->size) {
 				fprintf(fp, "%%CSIZE%%\n"
-								"%jd\n\n", (intmax_t)info->size);
+								"%"PRId64"\n\n", info->size);
 			}
 			if(info->isize) {
 				fprintf(fp, "%%ISIZE%%\n"
-								"%jd\n\n", (intmax_t)info->isize);
+								"%"PRId64"\n\n", info->isize);
 			}
 			if(info->md5sum) {
 				fprintf(fp, "%%MD5SUM%%\n"
diff --git a/lib/libalpm/delta.c b/lib/libalpm/delta.c
index 523968e..fc4afd1 100644
--- a/lib/libalpm/delta.c
+++ b/lib/libalpm/delta.c
@@ -22,7 +22,7 @@
 
 #include <stdlib.h>
 #include <string.h>
-#include <stdint.h> /* intmax_t */
+#include <inttypes.h> /* PRId64 */
 #include <limits.h>
 #include <sys/types.h>
 #include <regex.h>
@@ -238,7 +238,7 @@ off_t _alpm_shortest_delta_path(alpm_list_t *deltas,
 	dijkstra(vertices);
 	bestsize = shortest_path(vertices, to, &bestpath);
 
-	_alpm_log(PM_LOG_DEBUG, "delta shortest-path search complete : '%jd'\n", (intmax_t)bestsize);
+	_alpm_log(PM_LOG_DEBUG, "delta shortest-path search complete : '%"PRId64"'\n", bestsize);
 
 	alpm_list_free_inner(vertices, _alpm_graph_free);
 	alpm_list_free(vertices);
@@ -297,7 +297,7 @@ pmdelta_t *_alpm_delta_parse(char *line)
 	tmp2 = tmp;
 	STRDUP(delta->to, tmp2, RET_ERR(PM_ERR_MEMORY, NULL));
 
-	_alpm_log(PM_LOG_DEBUG, "delta : %s %s '%lld'\n", delta->from, delta->to, (long long)delta->delta_size);
+	_alpm_log(PM_LOG_DEBUG, "delta : %s %s '%"PRId64"'\n", delta->from, delta->to, delta->delta_size);
 
 	return(delta);
 }
diff --git a/lib/libalpm/dload.c b/lib/libalpm/dload.c
index 05555f2..717954f 100644
--- a/lib/libalpm/dload.c
+++ b/lib/libalpm/dload.c
@@ -271,8 +271,8 @@ static int download_internal(const char *url, const char *localpath,
 
 	if (ust.size != -1 && dl_thisfile < ust.size) {
 		pm_errno = PM_ERR_RETRIEVE;
-		_alpm_log(PM_LOG_ERROR, _("%s appears to be truncated: %jd/%jd bytes\n"),
-				filename, (intmax_t)dl_thisfile, (intmax_t)ust.size);
+		_alpm_log(PM_LOG_ERROR, _("%s appears to be truncated: %"PRId64"/%"PRId64" bytes\n"),
+				filename, dl_thisfile, ust.size);
 		ret = -1;
 		goto cleanup;
 	}
diff --git a/lib/libalpm/sync.c b/lib/libalpm/sync.c
index 2cdcd47..e3155fc 100644
--- a/lib/libalpm/sync.c
+++ b/lib/libalpm/sync.c
@@ -28,7 +28,7 @@
 #include <stdio.h>
 #include <fcntl.h>
 #include <string.h>
-#include <stdint.h> /* intmax_t */
+#include <inttypes.h> /* PRId64 */
 #include <unistd.h>
 #include <time.h>
 #include <dirent.h>
@@ -389,8 +389,8 @@ static int compute_download_size(pmpkg_t *newpkg)
 		size = alpm_pkg_get_size(newpkg);
 	}
 
-	_alpm_log(PM_LOG_DEBUG, "setting download size %jd for pkg %s\n",
-			(intmax_t)size, alpm_pkg_get_name(newpkg));
+	_alpm_log(PM_LOG_DEBUG, "setting download size %"PRId64" for pkg %s\n",
+			size, alpm_pkg_get_name(newpkg));
 
 	newpkg->download_size = size;
 	return(0);
-- 
1.6.5.2



More information about the pacman-dev mailing list