[Bug localedata/29023] New: ti_ER locale can cause confusion for parser

[Bug localedata/29023] New: ti_ER locale can cause confusion for parser

[euloanty at live dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29023

            Bug ID: 29023
           Summary: ti_ER locale can cause confusion for parser
           Product: glibc
           Version: unspecified
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: localedata
          Assignee: unassigned at sourceware dot org
          Reporter: euloanty at live dot com
                CC: libc-locales at sourceware dot org
  Target Milestone: ---

https://github.com/bminor/glibc/blob/master/localedata/locales/ti_ER

name_mr     "<U12A3><U1276>"             % "<U12A0><U12ED><U1270>" is informal

It makes my parse harder.
I think this should be changed to

name_mr     "<U12A3><U1276>"
% "<U12A0><U12ED><U1270>" is informal

Please put % into a separate line. Thank you!

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/29023] ti_ER locale can cause confusion for parser

[euloanty at live dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29023

--- Comment #1 from cqwrteur <euloanty at live dot com> ---
And ti_ET has the same issue.

https://github.com/bminor/glibc/blob/master/localedata/locales/ti_ET

name_mr     "<U12A0><U1276>"             % "<U12A0><U12ED><U1270>" is informal

I propose to change them to

name_mr     "<U12A0><U1276>"
% "<U12A0><U12ED><U1270>" is informal

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/29023] ti_ER locale can cause confusion for parser

[euloanty at live dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29023

cqwrteur <euloanty at live dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |euloanty at live dot com

--- Comment #2 from cqwrteur <euloanty at live dot com> ---
Created attachment 14048
  --> https://sourceware.org/bugzilla/attachment.cgi?id=14048&action=edit
diff

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/29023] ti_ER locale can cause confusion for parser

[schwab@linux-m68k.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=29023

--- Comment #3 from Andreas Schwab <schwab@linux-m68k.org> ---
Inline comments are common in locale definition files.  A parser will have to
cope.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/29023] ti_ER locale can cause confusion for parser

[euloanty at live dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29023

--- Comment #4 from cqwrteur <euloanty at live dot com> ---
(In reply to Andreas Schwab from comment #3)
> Inline comments are common in locale definition files.  A parser will have
> to cope.

but how the parse is going to know here?
it cannot differentiate whether it is string or part of string.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/29023] ti_ER locale can cause confusion for parser

[carlos at redhat dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29023

Carlos O'Donell <carlos at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |carlos at redhat dot com
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |NOTABUG

--- Comment #5 from Carlos O'Donell <carlos at redhat dot com> ---
(In reply to cqwrteur from comment #4)
> (In reply to Andreas Schwab from comment #3)
> > Inline comments are common in locale definition files.  A parser will have
> > to cope.
> 
> but how the parse is going to know here?
> it cannot differentiate whether it is string or part of string.

The operand is a string, as defined in ISO 30112 ('name_mr' definition), and
the definition ends after the closing double quote which encases the string.

Please adjust the parser to account for inline comments after the closing
double quote that terminates the string.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug localedata/29023] ti_ER locale can cause confusion for parser

[carlos at redhat dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29023

--- Comment #6 from Carlos O'Donell <carlos at redhat dot com> ---
Please refer to ISO 30112 for the additional grammar syntax.

Examples included in the standard:
<U0165> /d036/d055 % the character codes are only examples

Related grammar examples:
EOL = end_of_line | comment end_of_line ;
end_of_line = <anything that makes an End Of Line (EOL) in the operating system
employed> ;
comment = comment_char char* ;

-- 
You are receiving this mail because:
You are on the CC list for the bug.