aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBjörn Gustavsson <[email protected]>2016-05-12 15:53:36 +0200
committerBjörn Gustavsson <[email protected]>2016-05-13 10:52:25 +0200
commit467c4e5d198e0c4b9262c8e150a572050071ad41 (patch)
tree3b2c4c604d4bfcbb447db59e26e80fa92d262c08
parent3a8638e89e90dd2cfd311e3b98541ca3d6ac2b96 (diff)
downloadotp-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.erl2
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;