fix some minor problems

thanks @MichaelHatherly for review :]

Author: exaexa

src/utilities.jl

@@ -190,16 +190,16 @@ function printmethod_format(buffer::IOBuffer, binding::String, args::Vector{Stri
     if method_length_over_limit(
             length(binding) +
             1 +
-            sum(length.(args)) +
-            sum(length.(kws)) +
+            sum(length, args; init = 0) +
+            sum(length, kws; init = 0) +
             2*max(0, length(args)-1) +
             2*length(kws) +
             1 +
             length(return_type))
 
         sep_delim = "\n"
         paren_delim = "\n"
-        indent = "  "
+        indent = "    "
     end
 
     print(buffer, binding)
@@ -356,7 +356,7 @@ function printmethod(buffer::IOBuffer, binding::Docs.Binding, func, method::Meth
             type = "::$t"
         end
 
-        string(arg)*type*suffix
+        "$arg$type$suffix"
     end
 
     rt = Base.return_types(func, typesig)

test/tests.jl

@@ -307,7 +307,7 @@ end
             @test occursin("\n```julia\n", str)
             if VERSION > v"1.3.0"
                 @test occursin("\nk_5(::Type{T<:Number}, x::String) -> String\n", str)
-                @test occursin("\nk_5(\n  ::Type{T<:Number},\n  x::String,\n  func::Union{Nothing, Function}\n) -> String\n", str)
+                @test occursin("\nk_5(\n    ::Type{T<:Number},\n    x::String,\n    func::Union{Nothing, Function}\n) -> String\n", str)
                 @test occursin("\n```\n", str)
             else
                 # TODO: remove this test when julia 1.0.0 support is dropped.
@@ -343,11 +343,11 @@ end
             str = String(take!(buf))
             @test occursin("\n```julia\n", str)
             if VERSION >= v"1.6" && VERSION < v"1.7"
-                @test occursin("\nk_7(\n  x::Union{Nothing, T} where T<:Integer\n) -> Union{Nothing, Integer}\n", str)
-                @test occursin("\nk_7(\n  x::Union{Nothing, T} where T<:Integer,\n  y::Integer\n) -> Union{Nothing, Integer}\n", str)
+                @test occursin("\nk_7(\n    x::Union{Nothing, T} where T<:Integer\n) -> Union{Nothing, Integer}\n", str)
+                @test occursin("\nk_7(\n    x::Union{Nothing, T} where T<:Integer,\n    y::Integer\n) -> Union{Nothing, Integer}\n", str)
             else
-                @test occursin("\nk_7(\n  x::Union{Nothing, T} where T<:Integer\n) -> Union{Nothing, T} where T<:Integer\n", str)
-                @test occursin("\nk_7(\n  x::Union{Nothing, T} where T<:Integer,\n  y::Integer\n) -> Union{Nothing, T} where T<:Integer\n", str)
+                @test occursin("\nk_7(\n    x::Union{Nothing, T} where T<:Integer\n) -> Union{Nothing, T} where T<:Integer\n", str)
+                @test occursin("\nk_7(\n    x::Union{Nothing, T} where T<:Integer,\n    y::Integer\n) -> Union{Nothing, T} where T<:Integer\n", str)
             end
             @test occursin("\n```\n", str)