[Bug libstdc++/64680] New: basic_regex::operator= does not reset flags

[Bug libstdc++/64680] New: basic_regex::operator= does not reset flags

[kariya_mitsuru at hotmail dot com]

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

            Bug ID: 64680
           Summary: basic_regex::operator= does not reset flags
           Product: gcc
           Version: 5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: libstdc++
          Assignee: unassigned at gcc dot gnu.org
          Reporter: kariya_mitsuru at hotmail dot com

Created attachment 34491
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34491&action=edit
g++ -v

The sample code below should not throw a regex_error.

========================= sample code =========================
#include <regex>

int main()
{
    std::regex re("[[:alnum:]]", std::regex_constants::basic);
    re = "\\w+";
}
===============================================================
cf. http://melpon.org/wandbox/permlink/lrD2Ia4urIPVgakK


According to the C++11 standard 28.8.3[re.regex.assign], operator=(const charT*
ptr) is equivalent to assign(ptr), and assign(ptr) calls assign(const char*
ptr, flag_type f = regex_constants::ECMAScript).

Since "\\w+" is a valid ECMAScript syntax, the sample code above should end
normally.

[Bug libstdc++/64680] basic_regex::operator= does not reset flags

[timshen at gcc dot gnu.org]

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

--- Comment #1 from Tim Shen <timshen at gcc dot gnu.org> ---
Author: timshen
Date: Thu Jan 22 05:07:03 2015
New Revision: 219987

URL: https://gcc.gnu.org/viewcvs?rev=219987&root=gcc&view=rev
Log:
    PR libstdc++/64680
    * include/bits/regex.h (basic_regex<>::basic_regex,
    basic_regex<>::operator=, basic_regex<>::imbue): Conform to the
    standard interface.
    * testsuite/28_regex/basic_regex/assign/char/cstring.cc: New testcase.

Modified:
    trunk/libstdc++-v3/ChangeLog
    trunk/libstdc++-v3/include/bits/regex.h
    trunk/libstdc++-v3/testsuite/28_regex/basic_regex/assign/char/cstring.cc

[Bug libstdc++/64680] basic_regex::operator= does not reset flags

[timshen at gcc dot gnu.org]

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

--- Comment #2 from Tim Shen <timshen at gcc dot gnu.org> ---
Author: timshen
Date: Tue Feb  3 09:01:36 2015
New Revision: 220366

URL: https://gcc.gnu.org/viewcvs?rev=220366&root=gcc&view=rev
Log:
    PR libstdc++/64680
    Backported from mainline
    2015-01-22  Tim Shen  <timshen@google.com>

    * include/bits/regex.h (basic_regex<>::basic_regex,
    basic_regex<>::operator=, basic_regex<>::imbue): Conform to the
    standard interface.
    * testsuite/28_regex/basic_regex/assign/char/cstring.cc: New testcase.

Modified:
    branches/gcc-4_9-branch/libstdc++-v3/ChangeLog
    branches/gcc-4_9-branch/libstdc++-v3/include/bits/regex.h
   
branches/gcc-4_9-branch/libstdc++-v3/testsuite/28_regex/basic_regex/assign/char/cstring.cc

[Bug libstdc++/64680] basic_regex::operator= does not reset flags

[timshen at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
                 CC|                            |timshen at gcc dot gnu.org
         Resolution|---                         |FIXED

--- Comment #3 from Tim Shen <timshen at gcc dot gnu.org> ---
Resolved.

[Bug libstdc++/64680] basic_regex::operator= does not reset flags

[redi at gcc dot gnu.org]

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

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

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