New submission from Carsten Grohmann <carstengrohm...@gmx.de>: Hi,
the documentation of the decimal module contains a small recipe called moneyfmt() for format decimal values. It's very usefull. I'd like to suggest a small improvement because the output is incorrect with given dp="." (default) and places=0. Example: >>> moneyfmt(decimal.Decimal('-0.02'), neg='<', trailneg='>', places=1) '<0.0>' >>> moneyfmt(decimal.Decimal('-0.02'), neg='<', trailneg='>', places=0) '<0.>' Change: --- moneyfmt.py 2011-01-03 13:56:32.774169788 +0100 +++ moneyfmt.py.new 2011-01-03 13:56:58.130165330 +0100 @@ -33,7 +33,8 @@ build(trailneg) for i in range(places): build(next() if digits else '0') - build(dp) + if places: + build(dp) if not digits: build('0') i = 0 What do you think about the change? Regrads, Carsten ---------- assignee: d...@python components: Documentation messages: 125164 nosy: cgrohmann, d...@python priority: normal severity: normal status: open title: Suppress adding decimal point for places=0 in moneyfmt() _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue10813> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com