Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extra comma in yaws file #3

Open
rsharvey opened this issue Mar 27, 2015 · 1 comment
Open

Extra comma in yaws file #3

rsharvey opened this issue Mar 27, 2015 · 1 comment

Comments

@rsharvey
Copy link

ERL: OTP 17.4
Exodm: master

On line 800 of the deps/yang/yang_validate.erl file there is what looks like an extra comma. Look after "none":

rule('type') ->
    {'type',identifier_ref,
     ?ALL(0,none‚,[
           ?Y(0,1,'fraction-digits'), %% decimal64
           ?Y(0,1,'range'),   %% numerical
           ?Y(0,1,'length'),
           ?Y(0,n,'pattern'),  %% string
           ?Y(0,n,'enum'),   %% enum
           ?Y(0,1,'path'),   %% leafref FIXED: RFC6020 - Errata ID 2949
           ?Y(0,1,'base'),   %% identityref
           ?Y(0,1,'require-instance'),  %% instance-identifier
           ?Y(0,n,'bit'),               %% bits
           ?Y(0,n,'type')               %% union
          ])};
@uwiger
Copy link
Member

uwiger commented May 25, 2015

It's actually not a regular comma, but still, it should not be there, and has been removed in

Feuerlabs/yang@6e05025

It only seems to matter in OTP 17, since it made the move over to UTF-8 encoding of source files.

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

No branches or pull requests

2 participants