Browse Source

Rename option

Takeru Ohta 6 years ago
parent
commit
0b9dc77de4
3 changed files with 8 additions and 8 deletions
  1. 2 2
      src/jsone.erl
  2. 5 5
      src/jsone_decode.erl
  3. 1 1
      test/jsone_decode_tests.erl

+ 2 - 2
src/jsone.erl

@@ -228,7 +228,7 @@
 
 -type decode_option() :: {object_format, tuple | proplist | map}
                        | {allow_ctrl_chars, boolean()}
-                       | disallow_invalid_utf8
+                       | reject_invalid_utf8
                        | {'keys', 'binary' | 'atom' | 'existing_atom' | 'attempt_atom'}
                        | common_option().
 %% `object_format': <br />
@@ -242,7 +242,7 @@
 %% - If the value is `true', strings which contain unescaped control characters will be regarded as a legal JSON string <br />
 %% - default: `false'<br />
 %%
-%% `disallow_invalid_utf8': <br />
+%% `reject_invalid_utf8': <br />
 %% - Rejects JSON strings which contain invalid UTF-8 byte sequences <br />
 %%
 %% `keys': <br />

+ 5 - 5
src/jsone_decode.erl

@@ -67,7 +67,7 @@
         {
           object_format=?DEFAULT_OBJECT_FORMAT :: tuple | proplist | map,
           allow_ctrl_chars=false :: boolean(),
-          disallow_invalid_utf8=false :: boolean(),
+          reject_invalid_utf8=false :: boolean(),
           keys=binary :: 'binary' | 'atom' | 'existing_atom' | 'attempt_atom',
           undefined_as_null=false :: boolean()
         }).
@@ -189,9 +189,9 @@ string(<<$\\, B/binary>>, Base, Start, Nexts, Buf, Opt) ->
         <<$u, Bin/binary>> -> unicode_string(Bin, Start, Nexts, <<Buf/binary, Prefix/binary>>, Opt);
         _                  -> ?ERROR(string, [<<$\\, B/binary>>, Base, Start, Nexts, Buf, Opt])
     end;
-string(<<_, Bin/binary>>, Base, Start, Nexts, Buf, Opt) when Opt?OPT.allow_ctrl_chars, not Opt?OPT.disallow_invalid_utf8 ->
+string(<<_, Bin/binary>>, Base, Start, Nexts, Buf, Opt) when Opt?OPT.allow_ctrl_chars, not Opt?OPT.reject_invalid_utf8 ->
     string(Bin, Base, Start, Nexts, Buf, Opt);
-string(<<C, Bin/binary>>, Base, Start, Nexts, Buf, Opt) when 16#20 =< C, not Opt?OPT.disallow_invalid_utf8 ->
+string(<<C, Bin/binary>>, Base, Start, Nexts, Buf, Opt) when 16#20 =< C, not Opt?OPT.reject_invalid_utf8 ->
     string(Bin, Base, Start, Nexts, Buf, Opt);
 string(<<_/utf8, Bin/binary>>, Base, Start, Nexts, Buf, Opt) when Opt?OPT.allow_ctrl_chars ->
     string(Bin, Base, Start, Nexts, Buf, Opt);
@@ -306,8 +306,8 @@ parse_option([{object_format,F}|T], Opt) when F =:= tuple; F =:= proplist; F =:=
     parse_option(T, Opt?OPT{object_format=F});
 parse_option([{allow_ctrl_chars,B}|T], Opt) when is_boolean(B) ->
     parse_option(T, Opt?OPT{allow_ctrl_chars=B});
-parse_option([disallow_invalid_utf8|T], Opt) ->
-    parse_option(T, Opt?OPT{disallow_invalid_utf8=true});
+parse_option([reject_invalid_utf8|T], Opt) ->
+    parse_option(T, Opt?OPT{reject_invalid_utf8=true});
 parse_option([{keys, K}|T], Opt)
   when K =:= binary; K =:= atom; K =:= existing_atom; K =:= attempt_atom ->
     parse_option(T, Opt?OPT{keys = K});

+ 1 - 1
test/jsone_decode_tests.erl

@@ -296,6 +296,6 @@ decode_test_() ->
       fun () ->
               Input = <<123,34,105,100,34,58,34,190,72,94,90,253,121,94,71,73,68,91,122,211,253,32,94,86,67,163,253,230,34,125>>,
               ?assertMatch({ok, _, _}, jsone:try_decode(Input)),
-              ?assertMatch({error, {badarg, _}}, jsone:try_decode(Input, [disallow_invalid_utf8]))
+              ?assertMatch({error, {badarg, _}}, jsone:try_decode(Input, [reject_invalid_utf8]))
       end}
     ].