Return nothing from printfmt

Author: ScottPJones

Diff for: src/fmtspec.jl

@@ -214,6 +214,7 @@ function printfmt(io::IO, fs::FormatSpec, x)
     else # cls == 'c'
         _pfmt_s(io, fs, Char(x))
     end
+    nothing
 end
 
 printfmt(fs::FormatSpec, x) = printfmt(_stdout(), fs, x)

Diff for: src/formatexpr.jl

@@ -151,6 +151,7 @@ function printfmt(io::IO, fe::FormatExpr, args...)
         end
     end
     isempty(fe.suffix) || print(io, fe.suffix)
+    nothing
 end
 
 const StringOrFE = Union{AbstractString, FormatExpr}

Diff for: test/fmtspec.jl

@@ -223,13 +223,13 @@ end
     # May need to change how separators are handled with zero padding,
     # to produce the same results as Python
 
-#    @test pyfmt("08,d",  1234) == "0,001,234"
-#    @test pyfmt("09,d",  1234) == "0,001,234"
-#    @test pyfmt("010,d", 1234) == "00,001,234"
+    @test_broken pyfmt("08,d",  1234) == "0,001,234"
+    @test_broken pyfmt("09,d",  1234) == "0,001,234"
+    @test_broken pyfmt("010,d", 1234) == "00,001,234"
 
     @test pyfmt("+#012_b", 42) == "+0b0010_1010"
-#    @test pyfmt("+#013_b", 42) == "+0b0_0010_1010"
-#    @test pyfmt("+#014_b", 42) == "+0b0_0010_1010"
+    @test_broken pyfmt("+#013_b", 42) == "+0b0_0010_1010"
+    @test_broken pyfmt("+#014_b", 42) == "+0b0_0010_1010"
 
 end
 

Diff for: test/formatexpr.jl

@@ -34,13 +34,17 @@
     @test_throws(ErrorException, format("{1", 10) )
 
     f = FormatExpr("{1:s}")
-    printfmt(f,"")
-    printfmtln(f,"")
+    printfmt(f, "")
+    printfmtln(f, "")
     io = IOBuffer()
-    printfmt(io,f,"foobar")
+    printfmt(io, f, "foobar")
     @test io.size == 6
-    printfmtln(io,f,"ugly")
+    printfmtln(io, f, "ugly")
     @test io.size == 11
+
+    # Issue #81
+    @test printfmt("{:d}", 1000) == nothing
+
 end
 
 @testset "format escape {{ and }}" begin