From: David M. <run...@us...> - 2011-11-12 10:27:15
|
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "pdl". The branch, master has been updated via 414c56330c2ed44d932bc510a9f5e25245c5a137 (commit) from e3b6377a771cc6be639e374553598e1c50fe4703 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit 414c56330c2ed44d932bc510a9f5e25245c5a137 Author: Tim Haines <com...@gm...> Date: Sat Nov 12 04:20:53 2011 -0600 Fixed numerous prinft errors. The source had many prinft format errors that were causing compilers to spew lots of warning messages. For details, see sf bug #3391645. Here is Tim's original email that included the patch: --------%<-------- Core.xs.PL, pdlapi.c, pdlthread.c, pdlconv.c.PL, pdlmagic.c: updated printf format flags for printing pointers pdlcore.c.PL: printf on line 920 had argument specifiers, but no arguments Core.xs.PL: ftruncate was compiled with attribute warn_unused_result, but return value was unchecked. I added a capture of the return value, and croaked if it was bad. I didn't parse the errno returned since I didn't want to taint the #include space with errno.h and string.h (I had seen some comments about strndup causing conflicts in one of the other files.) I am also getting the following warnings for Lib/Slatec/slatec/snrm2.f. snrm2.f:87.72: 10 ASSIGN 30 TO NEXT 1 Warning: Deleted feature: ASSIGN statement at (1) slatec/snrm2.f:94.19: 20 GO TO NEXT,(30, 50, 70, 110) 1 Warning: Deleted feature: Assigned GOTO statement at (1) slatec/snrm2.f:96.72: ASSIGN 50 TO NEXT 1 Warning: Deleted feature: ASSIGN statement at (1) slatec/snrm2.f:106.72: ASSIGN 70 TO NEXT 1 Warning: Deleted feature: ASSIGN statement at (1) slatec/snrm2.f:112.72: ASSIGN 110 TO NEXT 1 Warning: Deleted feature: ASSIGN statement at (1) I don't know much about Fortran, so I didn't change any of these. I did a successful build and test after making the modifications above using gcc 4.5.2 and glibc 2.13 on Ubuntu 11.04. -------->%-------- Signed-off-by: David Mertens <dcm...@gm...> ----------------------------------------------------------------------- Summary of changes: Basic/Core/Core.xs.PL | 21 ++++++++-- Basic/Core/pdlapi.c | 104 +++++++++++++++++++++++----------------------- Basic/Core/pdlconv.c.PL | 2 +- Basic/Core/pdlcore.c.PL | 2 +- Basic/Core/pdlmagic.c | 2 +- Basic/Core/pdlthread.c | 16 ++++---- 6 files changed, 80 insertions(+), 67 deletions(-) hooks/post-receive -- pdl |