瀏覽代碼

Remove the workaround for the bug when printing empty binaries in W mode

The fix for this was merged in R16 somewhere, so this workaround is no
longer needed, and fixes the EQC test from spotting the divergent
behaviour with io_lib.

See also: http://erlang.org/pipermail/erlang-patches/2012-September/003034.html
pull/192/head
Andrew Thompson 11 年之前
父節點
當前提交
4c7bd96d62
共有 1 個檔案被更改,包括 5 行新增8 行删除
  1. +5
    -8
      src/lager_trunc_io.erl

+ 5
- 8
src/lager_trunc_io.erl 查看文件

@ -127,13 +127,10 @@ print(Atom, _Max, #print_options{force_strings=NoQuote}) when is_atom(Atom) ->
end, end,
{R, length(R)}; {R, length(R)};
print(Bin, _Max, O = #print_options{depth=1}) when is_binary(Bin) ->
case O#print_options.lists_as_strings of
true when Bin == <<>> ->
{"<<>>", 4};
_ ->
{"<<...>>", 7}
end;
print(<<>>, _Max, #print_options{depth=1}) ->
{"<<>>", 4};
print(Bin, _Max, #print_options{depth=1}) when is_binary(Bin) ->
{"<<...>>", 7};
print(<<>>, _Max, Options) -> print(<<>>, _Max, Options) ->
case Options#print_options.force_strings of case Options#print_options.force_strings of
true -> true ->
@ -766,7 +763,7 @@ depth_limit_test() ->
%% I don't even know... %% I don't even know...
?assertEqual("<<>>", lists:flatten(format("~P", [<<>>, 1], 50))), ?assertEqual("<<>>", lists:flatten(format("~P", [<<>>, 1], 50))),
?assertEqual("<<...>>", lists:flatten(format("~W", [<<>>, 1], 50))),
?assertEqual("<<>>", lists:flatten(format("~W", [<<>>, 1], 50))),
?assertEqual("{abc,<<\"abc\\\"\">>}", lists:flatten(format("~P", [{abc,<<"abc\"">>}, 4], 50))), ?assertEqual("{abc,<<\"abc\\\"\">>}", lists:flatten(format("~P", [{abc,<<"abc\"">>}, 4], 50))),

Loading…
取消
儲存