more type stability for prune_json

Author: racinmat

Diff for: src/schema/schema.jl

@@ -58,9 +58,5 @@ prune_json(json, sch::ArrayEntry) = map(json) do el
 end
 
 function prune_json(json, sch::DictEntry)
-    out = Dict()
-    for (k,v) in children(sch)
-        String(k) ∈ keys(json) && (out[String(k)] = prune_json(json[String(k)], v))
-    end
-    out
+    Dict(String(k) => prune_json(json[String(k)], v) for (k,v) in children(sch) if String(k) ∈ keys(json))
 end

Diff for: test/schema.jl

@@ -502,30 +502,30 @@ end
 
 	sch = JsonGrinder.schema([j1,j2,j3,j4,j5,j6])
 
-	@test JsonGrinder.prune_json(j1, sch) == Dict(
+	@test JsonGrinder.prune_json(j1, sch) == Dict{String, Any}(
 		"c" => Dict("a"=>Dict("b"=>[4.0, 5.0, 6.0],"a"=>[1.0, 2.0, 3.0])),
 		"b" => Dict("b"=>1.0,"a"=>[1.0, 2.0, 3.0]),
 		"a" => 4.0)
 
-	@test JsonGrinder.prune_json(j2, sch) == Dict(
+	@test JsonGrinder.prune_json(j2, sch) == Dict{String, Any}(
 		"c" => Dict("a"=>Dict("b"=>[5, 6],"a"=>[2, 3])),
 		"a" => 4)
 
 	delete!(sch.childs, :b)
 
-	@test JsonGrinder.prune_json(j1, sch) == Dict(
+	@test JsonGrinder.prune_json(j1, sch) == Dict{String, Any}(
   		"c" => Dict("a"=>Dict("b"=>[4.0, 5.0, 6.0],"a"=>[1.0, 2.0, 3.0])),
   		"a" => 4.0)
 
-	@test JsonGrinder.prune_json(j2, sch) == Dict(
+	@test JsonGrinder.prune_json(j2, sch) == Dict{String, Any}(
 		"c" => Dict("a"=>Dict("b"=>[5, 6],"a"=>[2, 3])),
 		"a" => 4)
 
 	delete!(sch.childs, :c)
 
-	@test JsonGrinder.prune_json(j1, sch) == Dict(
+	@test JsonGrinder.prune_json(j1, sch) == Dict{String, Any}(
   		"a" => 4.0)
 
-	@test JsonGrinder.prune_json(j2, sch) == Dict(
+	@test JsonGrinder.prune_json(j2, sch) == Dict{String, Any}(
 		"a" => 4)
 end