Hi, Haven't heard any opinions on these functionality changes... (Though I know more significant things keep you busy!)
Do many other people use number_format()?? Wouldn't allowing thousands separator as param #2 simplify things? Any issues with the semantics of operation? Negative precision to specify trailing zero removal seems fine at least, even though that's different than round()'s behavior. I've now made some tests, including changes of course, in case you were just waiting for them? Hope they're enough. http://realplain.com/php/number_format.phpt Thanks, Matt ----- Original Message ----- From: "Matt W" Sent: Wednesday, July 19, 2006 > Hi, > > There are a couple things I don't like about how number_format() works > currently, so I'm hoping the functionality changes this patch provides can > be added. :-) > > First is that when you're working with whole numbers and/or don't want any > decimal places, both decimal-related parameters are needed before the > thousands separator. I always thought it would be logical to pass the > thousands separator as param #2 in those cases. That's what my changes > allow if it's a non-numeric string. > > Second is that when decimals places ARE specified, I would like to be able > to have trailing zeros/point removed. (Like "(string) round(...)" but never > scientific notation.) I'm sure I'm not the only one that has wanted grouped > thousands and/or custom decimal point *without* trailing 0's. :-) The patch > allows this by specifying negative decimal places -- e.g. -2 is 2 decimals > without extra zeros. > > Are these enhancements all right? If they're agreed to, I'll create more > number_format() tests right away if desired. Just wanted to get any > thoughts on the patch first. > > Also, I changed PHP_ROUND_WITH_FUZZ to skip unnecessary steps if places==0 > (round() is nearly 20% faster then, in my quick tests), corrected the > misspelling of "separator" in the proto, and simplified some of the code. > > http://realplain.com/php/number_format_enhancements.diff > http://realplain.com/php/number_format_enhancements_5_2.diff > > > Thanks, > Matt -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
