From 53a5eb4773ed7bf9df12d0be7781a0e70b458e04 Mon Sep 17 00:00:00 2001 From: Arnaud Giersch Date: Tue, 19 Nov 2013 21:43:33 +0100 Subject: [PATCH] Add some words about integer typesi in README.coding. --- README.coding | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/README.coding b/README.coding index 35c9a55726..f03cc78b23 100644 --- a/README.coding +++ b/README.coding @@ -92,7 +92,7 @@ bug. Please report it (or fix it yourself if you can). ** ***************************************************** -MALLOC: +MALLOC Don't use it, or you'll have to check the result (and do some dirty stuff on AIX). Use xbt_malloc (or even better, xbt_new) instead. @@ -102,6 +102,12 @@ SIZE_T (FIXME: obsolete?) do cast it to unsigned long before printing (and use %lu), or use %zu. +INTEGERS + Please avoid to use long ints. This is the source of many compatibility + problems betwwen 32 bits and 64 bits archs. Either use plain ints (generally + 32 bits width) or long long ints (64 bits width, at least). At last resort + consider using integer types defined in C99 by . + PRINTF pointer difference (FIXME: advertise %td instead?) printf ("diff = %ld\n", (long) (pointer2 - pointer1)); -- 2.20.1