ID: 48342 User updated by: jf at netmadeira dot com Reported By: jf at netmadeira dot com Status: Wont fix Bug Type: Compile Warning Operating System: linux PHP Version: 5.3CVS-2009-05-20 (snap) New Comment:
But wont it crash php if: case 'Y': length = slprintf(buffer, 32, "%s%04ld", t->y < 0 ? "-" : "", llabs((timelib_sll) t->y)); break; is called ?? Sorry, didn't understand what you mean when you told to see bug #45188 I'm not native englisg speaker.. Best regards Previous Comments: ------------------------------------------------------------------------ [2009-05-20 11:25:09] [email protected] I forgot: # man llabs And also see bug #45188 (where the compile failure was reported in last comments) ------------------------------------------------------------------------ [2009-05-20 08:19:23] [email protected] a) It's harmless b) "Fixing" it breaks stuff with older glibc ------------------------------------------------------------------------ [2009-05-20 05:01:35] jf at netmadeira dot com Description: ------------ Received this compile warming while compilling: (...)/php_date.c: In function 'date_format': (...)/php_date.c:1040: warning: incompatible implicit declaration of built-in function 'llabs' So did a grep -R llabs ./ and get: ./win32/php_stdint.h:static __inline int64_t llabs( int64_t i ) { return i >= 0? i: -i; } ./ext/date/php_date.c: case 'Y': length = slprintf(buffer, 32, "%s%04ld", t->y < 0 ? "-" : "", llabs((timelib_sll) t->y)); break; ./ext/fileinfo/tests/magic:# from Ruda Moura <[email protected]> it seems llabs its only available for windows, is this right ?? Can I filter all warmings, I spotted this one because I canceled compilation right in begining and seen that, but in midle of all those lines can't spot nothing more.. Regards Reproduce code: --------------- grep -R llabs ./ in php package source. ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=48342&edit=1
