diff options
author | Björn Gustavsson <[email protected]> | 2015-01-27 13:42:31 +0100 |
---|---|---|
committer | Björn Gustavsson <[email protected]> | 2015-01-28 11:59:53 +0100 |
commit | 440c54f6a6b78015e8c8e0c7d16ad3cbb3d22147 (patch) | |
tree | 232502616859bd1c428c4f17cc867bc27c7faff0 /lib/compiler/test | |
parent | 4a3ad317ec6ebe02ad1ecc8a4132896c333ba742 (diff) | |
download | otp-440c54f6a6b78015e8c8e0c7d16ad3cbb3d22147.tar.gz otp-440c54f6a6b78015e8c8e0c7d16ad3cbb3d22147.tar.bz2 otp-440c54f6a6b78015e8c8e0c7d16ad3cbb3d22147.zip |
core_pp: Correct printing of map literals
A map key in a pattern would be incorrectly pretty-printed.
As an example, the pattern in:
x() ->
#{ #{ a => 3 } := 42 } = X.
would be pretty-printed as:
<~{~<~{~<'a',3>}~,42>}~
instead of:
<~{~<~{::<'a',3>}~,42>}~
When this problem has been corrected, the workaround for it in
cerl:ann_c_map/3 can be removed. The workaround was not harmless,
as it would cause the following map update to incorrectly succeed:
(#{})#{a:=1}
Diffstat (limited to 'lib/compiler/test')
-rw-r--r-- | lib/compiler/test/map_SUITE.erl | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/lib/compiler/test/map_SUITE.erl b/lib/compiler/test/map_SUITE.erl index dbac61765b..bc5ae803c6 100644 --- a/lib/compiler/test/map_SUITE.erl +++ b/lib/compiler/test/map_SUITE.erl @@ -319,6 +319,12 @@ t_update_exact(Config) when is_list(Config) -> {'EXIT',{badarg,_}} = (catch M0#{42=>v1,42.0:=v2,42:=v3}), {'EXIT',{badarg,_}} = (catch <<>>#{nonexisting:=val}), {'EXIT',{badarg,_}} = (catch M0#{<<0:257>> := val}), %% limitation + + %% A workaround for a bug allowed an empty map to be updated. + {'EXIT',{badarg,_}} = (catch (id(#{}))#{a:=1}), + {'EXIT',{badarg,_}} = (catch #{}#{a:=1}), + Empty = #{}, + {'EXIT',{badarg,_}} = (catch Empty#{a:=1}), ok. t_update_values(Config) when is_list(Config) -> |