Merge pull request #47 from jmkuhn/pull-request/d2df85fa

Round before calculating length (Fixes #46)

Author: jmkuhn

src/fmtcore.jl

@@ -149,10 +149,6 @@ function _pfmt_float(out::IO, sch::Char, zs::Integer, intv::Real, decv::Real, pr
         _repwrite(out, '0', zs)
     end
     idecv = round(Integer, decv * exp10(prec))
-    if idecv == exp10(prec)
-        intv += 1
-        idecv = 0
-    end
     # print integer part
     if intv == 0
         write(out, '0')
@@ -173,10 +169,10 @@ end
 
 function _pfmt_f(out::IO, fs::FormatSpec, x::AbstractFloat)
     # separate sign, integer, and decimal part
-    ax = abs(x)
+    rax = round(abs(x), fs.prec)
     sch = _signchar(x, fs.sign)
-    intv = trunc(Integer, ax)
-    decv = ax - intv
+    intv = trunc(Integer, rax)
+    decv = rax - intv
 
     # calculate length
     xlen = _ndigits(intv, _Dec()) + 1 + fs.prec

test/fmtspec.jl

@@ -143,6 +143,7 @@ fs = FormatSpec("d")
 
 @test fmt(".2f", 0.999) == "1.00"
 @test fmt(".2f", 0.996) == "1.00"
+@test fmt("6.2f", 9.999) == " 10.00"
 # Floating point error can upset this one (i.e. 0.99500000 or 0.994999999)
 @test (fmt(".2f", 0.995) == "1.00" || fmt(".2f", 0.995) == "0.99")
 @test fmt(".2f", 0.994) == "0.99"