diff options
author | Björn Gustavsson <[email protected]> | 2016-05-12 15:53:36 +0200 |
---|---|---|
committer | Björn Gustavsson <[email protected]> | 2016-05-13 10:52:25 +0200 |
commit | 467c4e5d198e0c4b9262c8e150a572050071ad41 (patch) | |
tree | 3b2c4c604d4bfcbb447db59e26e80fa92d262c08 | |
parent | 3a8638e89e90dd2cfd311e3b98541ca3d6ac2b96 (diff) | |
download | otp-467c4e5d198e0c4b9262c8e150a572050071ad41.tar.gz otp-467c4e5d198e0c4b9262c8e150a572050071ad41.tar.bz2 otp-467c4e5d198e0c4b9262c8e150a572050071ad41.zip |
core_pp: Remove uncovered clause in is_simple_term/1
The clause for handling #c_values{} in is_simple_term/1 is never
executed. It can be safely removed, since there is a default clause
that will return 'false' in the extremly unlikely event that a
Without the clause, code such as:
let <_v1,_v2> = <1,2>
in {_v1,_v2}
would be printed with an extra newline:
let <_v1,_v2> =
<1,2>
in {_v1,_v2}
-rw-r--r-- | lib/compiler/src/core_pp.erl | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/lib/compiler/src/core_pp.erl b/lib/compiler/src/core_pp.erl index 22171a70c7..67209d06be 100644 --- a/lib/compiler/src/core_pp.erl +++ b/lib/compiler/src/core_pp.erl @@ -518,8 +518,6 @@ add_indent(Ctxt, Dx) -> core_atom(A) -> io_lib:write_string(atom_to_list(A), $'). -is_simple_term(#c_values{es=Es}) -> - length(Es) < 3 andalso lists:all(fun is_simple_term/1, Es); is_simple_term(#c_tuple{es=Es}) -> length(Es) < 4 andalso lists:all(fun is_simple_term/1, Es); is_simple_term(#c_var{}) -> true; |