From: Subrata <sub...@us...> - 2009-11-02 13:57:56
|
Update of /cvsroot/ltp/ltp/testcases/kernel/syscalls/move_pages In directory ddv4jf1.ch3.sourceforge.com:/tmp/cvs-serv23367/ltp/testcases/kernel/syscalls/move_pages Modified Files: move_pages_support.c Log Message: Other compiler warning fixes: this is another chunk of compiler warning fixes in LTP tests, related to printf format strings. They have multiple causes: - most of them are caused by %d used for the TEST_RETURN variable (as in a previous patch), - off_t are cast to (int64_t) and PRId64 output specifier is used for them, - other types are cast to (intmax_t) or (uintmax_t) and use the %jd or %ju (C99) output specifier. I've used this solution because it's generally correct, easy to use solution that is portable and (IMHO) more readable than using PRId64/PRIu64 and a cast (which you generally can't avoid). This way, there's only a cast, - in ioperm test, address should be declared as unsigned long (according to ioperm(2)), - other little fixes (foo vs. *foo, %S vs. %s, %0 vs. %o). Signed-off-by: Jiri Palecek <jpa...@we...>. Index: move_pages_support.c =================================================================== RCS file: /cvsroot/ltp/ltp/testcases/kernel/syscalls/move_pages/move_pages_support.c,v retrieving revision 1.3 retrieving revision 1.4 diff -C2 -d -r1.3 -r1.4 *** move_pages_support.c 9 Oct 2009 17:55:58 -0000 1.3 --- move_pages_support.c 2 Nov 2009 13:57:17 -0000 1.4 *************** *** 190,194 **** if (status[i] != nodes[i]) { tst_resm(TFAIL, "page %d on node %d, " ! "expected on node %p", i, status[i], nodes[i]); return; } --- 190,194 ---- if (status[i] != nodes[i]) { tst_resm(TFAIL, "page %d on node %d, " ! "expected on node %d", i, status[i], nodes[i]); return; } |