Skip to content

Conversation

@kianmeng
Copy link

List of additional changes:

  • prettify README
  • resolve compiler warnings from tests
  • resolve dialyzer warnings
  • use ex_doc for doc generation

List of additional changes:
- prettify README
- resolve compiler warnings from tests
- resolve dialyzer warnings
- use ex_doc for doc generation
%% API
-export([to_ascii_test/0, to_unicode_test/0]).


Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$ rebar3 eunit
===> Verifying dependencies...
===> Analyzing applications...
===> Compiling idna
   ┌─ test/punycode_test.erl:
   │
 8 │  -export([punicode_encode_test/0, punicode_decode_test/0]).
   │   ╰── Warning: function punicode_decode_test/0 already exported

   ┌─ test/punycode_test.erl:
   │
 8 │  -export([punicode_encode_test/0, punicode_decode_test/0]).
   │   ╰── Warning: function punicode_encode_test/0 already exported


   ┌─ test/compat_test.erl:
   │
 9 │  -export([to_ascii_test/0, to_unicode_test/0]).
   │   ╰── Warning: function to_ascii_test/0 already exported

   ┌─ test/compat_test.erl:
   │
 9 │  -export([to_ascii_test/0, to_unicode_test/0]).
   │   ╰── Warning: function to_unicode_test/0 already exported


     ┌─ test/uts46_test.erl:
     │
 127 │  to_unicode(S) ->
     │  ╰── Warning: function to_unicode/1 is unused


===> Performing EUnit tests...
...............

Dir = idna_data:bidirectional(C),
case lists:member(Dir, ["R", "AL", "AN"]) of
true -> true;
false -> check_rtl(Rest, Label)
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rebar3 dialyzer
===> Verifying dependencies...
===> Analyzing applications...
===> Compiling idna
===> Dialyzer starting, this may take a while...
===> Add debug_info to compiler options (erl_opts) if Dialyzer fails to load Core Erlang.
===> Updating plt...
===> Resolving project files...
===> Updating base plt...
===> Resolving base files...
===> Checking 222 files in /home/ang/.cache/rebar3/rebar3_28.1_plt...
===> Copying /home/ang/.cache/rebar3/rebar3_28.1_plt to /home/ang/src/contrib/eef/erlang-idna/_build/default/rebar3_28.1_plt...
===> Checking 222 files in _build/default/rebar3_28.1_plt...
===> Doing success typing analysis...
===> Resolving project warning files...
===> Analyzing 8 files with _build/default/rebar3_28.1_plt...

src/idna.erl
Line 373 Column 1: The pattern <Char, <<>>> can never match the type <string() | char(),[any()]>
Line 374 Column 1: The pattern <Char, <<>>> can never match the type <string() | char(),[any()]>
Line 375 Column 24: Guard test is_binary(Bin::[any()]) can never succeed

src/idna_bidi.erl
Line 28 Column 5: The pattern 'false' can never match the type [1..255,...]
===> Warnings written to _build/default/28.1.dialyzer_warnings
===> Warnings occurred running dialyzer: 4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant