ID: 28633 Updated by: [EMAIL PROTECTED] Reported By: jschmidt at thriftycmh dot net -Status: Assigned +Status: Closed -Bug Type: Math related +Bug Type: *Mail Related Operating System: Linux 2.4 PHP Version: 4.3.7 Assigned To: iliaa New Comment:
This bug has been fixed in CVS. Snapshots of the sources are packaged every three hours; this change will be in the next snapshot. You can grab the snapshot at http://snaps.php.net/. Thank you for the report, and for helping us make PHP better. Previous Comments: ------------------------------------------------------------------------ [2004-07-06 01:33:36] [EMAIL PROTECTED] Ilia, recheck.. :) ------------------------------------------------------------------------ [2004-06-16 22:36:13] php at alterego dot dp dot ua Sorry, but this actually is a bug. Kind of brain damage. echo sprintf('%05.2f', 0.02) . "\n"; 00000.02 echo sprintf('%05.2f', 2.02) . "\n"; 02.02 Although the double-checked documentation is very uncertain about how this should work, it definitely should work one of the ways, not both. ------------------------------------------------------------------------ [2004-06-04 22:26:26] jschmidt at thriftycmh dot net Actually, I did find inconsistency (bug) with the new behavior. echo sprintf("%06.2f",'1.00'); 001.00 echo sprintf("%06.2f",'.01'); 000000.01 So which is it? Pad to 6 characters or pad to 6 on the left of the decimal point? ------------------------------------------------------------------------ [2004-06-04 18:47:23] jschmidt at thriftycmh dot net OK, looking at /ext/standard formatted_print.c I see what has changed. Before we were specifying the desired number of characters to pad to the left of the decimal. Now we are specifying the total desired number of characters the output should be padded to. So technically, we are indeed "padding the results to the right string size". Too bad it has been the previous way for so long. ------------------------------------------------------------------------ [2004-06-04 17:56:29] jschmidt at thriftycmh dot net How can this be correct behavior if it was working this way for all the previous 4.3.X versions? I did check the manual and I see no updates to indicate that this is changed behavior. In fact, quoting from the manual: http://www.php.net/manual/en/function.sprintf.php "Each conversion specification consists of a percent sign (%), followed by one or more of these elements, in order: An optional padding specifier that says what character will be used for padding the results to the right string size. This may be a space character or a 0 (zero character). The default is to pad with spaces" So again, sprintf("%3.2f",'1.00') is no longer padding with leading spaces as it was in previous versions. If it is a changed behavior, please indicate this in the documentation. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/28633 -- Edit this bug report at http://bugs.php.net/?id=28633&edit=1
