[Bug c++/63775] New: [C++11] Regex range with leading dash (-) not working

[Bug c++/63775] New: [C++11] Regex range with leading dash (-) not working

[gnu-org at bignm dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63775

            Bug ID: 63775
           Summary: [C++11] Regex range with leading dash (-) not working
           Product: gcc
           Version: 4.9.1
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
          Assignee: unassigned at gcc dot gnu.org
          Reporter: gnu-org at bignm dot com

Created attachment 33918
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33918&action=edit
Source code of test program

Partial output from file below:

Note the String below should match the Regexp. Full source code of the test
program is attached below.

DEBUG: test_group(): String = 'module_method'
DEBUG: test_group(): Regexp = '/^([-a-z0-9]+)_([-a-z0-9]+)$/o'
DEBUG: group_regexp(): Using 'o' flag
DEBUG: group_regexp(): Match Failed!
DEBUG: test_group(): First match in 0.000753880 seconds
DEBUG: test_group(): First match failed!
DEBUG: group_regexp(): Using 'o' flag
DEBUG: group_regexp(): Match Failed!
DEBUG: test_group(): Second match in 0.000759125 seconds
DEBUG: test_group(): Second match failed!

[Bug libstdc++/63775] [C++11] Regex range with leading dash (-) not working

[timshen at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63775

Tim Shen <timshen at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |timshen at gcc dot gnu.org

--- Comment #1 from Tim Shen <timshen at gcc dot gnu.org> ---
Ah, the current bracket expression implementation is too naive. I'll fix it.

Thanks for reporting :)

[Bug libstdc++/63775] [C++11] Regex range with leading dash (-) not working

[timshen at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63775

--- Comment #2 from Tim Shen <timshen at gcc dot gnu.org> ---
Author: timshen
Date: Thu Nov 13 07:40:01 2014
New Revision: 217461

URL: https://gcc.gnu.org/viewcvs?rev=217461&root=gcc&view=rev
Log:
    PR libstdc++/63775
    * include/bits/regex_compiler.h (_Compiler<>::_M_expression_term,
    _BracketMatcher<>::_M_make_range): Throw regex_erorr on invalid range
    like [z-a]. Change _M_expression_term interface.
    * include/bits/regex_compiler.tcc (
    _Compiler<>::_M_insert_bracket_matcher,
    _Compiler<>::_M_expression_term): Rewrite bracket expression parsing.
    * testsuite/28_regex/algorithms/regex_match/cstring_bracket_01.cc:
    Add testcases and move file out of extended.

Added:
   
trunk/libstdc++-v3/testsuite/28_regex/algorithms/regex_match/cstring_bracket_01.cc
Removed:
   
trunk/libstdc++-v3/testsuite/28_regex/algorithms/regex_match/extended/cstring_bracket_01.cc
Modified:
    trunk/libstdc++-v3/ChangeLog
    trunk/libstdc++-v3/include/bits/regex_compiler.h
    trunk/libstdc++-v3/include/bits/regex_compiler.tcc

[Bug libstdc++/63775] [C++11] Regex range with leading dash (-) not working

[timshen at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63775

--- Comment #4 from Tim Shen <timshen at gcc dot gnu.org> ---
Author: timshen
Date: Fri Jun  5 04:58:26 2015
New Revision: 224144

URL: https://gcc.gnu.org/viewcvs?rev=224144&root=gcc&view=rev
Log:
        PR libstdc++/66359
        Backport from mainline
        2014-11-13  Tim Shen  <timshen@google.com>

        PR libstdc++/63775
        * include/bits/regex_compiler.h (_Compiler<>::_M_expression_term,
        _BracketMatcher<>::_M_make_range): Throw regex_erorr on invalid range
        like [z-a]. Change _M_expression_term interface.
        * include/bits/regex_compiler.tcc (
        _Compiler<>::_M_insert_bracket_matcher,
        _Compiler<>::_M_expression_term): Rewrite bracket expression parsing.
        * testsuite/28_regex/algorithms/regex_match/cstring_bracket_01.cc:
        Add testcases and move file out of extended.

Added:
   
branches/gcc-4_9-branch/libstdc++-v3/testsuite/28_regex/algorithms/regex_match/cstring_bracket_01.cc
Removed:
   
branches/gcc-4_9-branch/libstdc++-v3/testsuite/28_regex/algorithms/regex_match/extended/cstring_bracket_01.cc
Modified:
    branches/gcc-4_9-branch/libstdc++-v3/ChangeLog
    branches/gcc-4_9-branch/libstdc++-v3/include/bits/regex_compiler.h
    branches/gcc-4_9-branch/libstdc++-v3/include/bits/regex_compiler.tcc

[Bug libstdc++/63775] [C++11] Regex range with leading dash (-) not working

[redi at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63775

Jonathan Wakely <redi at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.9.3