diff mbox

Support embedded zeros in sub_match comparisons (DR 2217)

Message ID CAG4ZjNmVc8rKn9PPoAXzvzhc-0cBOJ3Pm2xjCj73J=NEsp1t0A@mail.gmail.com
State New
Headers show

Commit Message

Tim Shen Nov. 14, 2014, 4:29 a.m. UTC
On Thu, Nov 13, 2014 at 3:47 PM, Jonathan Wakely <jwakely@redhat.com> wrote:
> Tim, I think Daniel still doesn't have commit access, so could you
> please commit this? (And CC the patch to gcc-patches).

Committed. Removed trailing spaces.

The svn diff is attached.

Thank you!
diff mbox

Patch

Index: include/bits/regex.h
===================================================================
--- include/bits/regex.h	(revision 217534)
+++ include/bits/regex.h	(revision 217535)
@@ -1006,7 +1006,10 @@ 
     inline bool
     operator==(const __sub_match_string<_Bi_iter, _Ch_traits, _Ch_alloc>& __lhs,
 	       const sub_match<_Bi_iter>& __rhs)
-    { return __rhs.compare(__lhs.c_str()) == 0; }
+    {
+      typedef typename sub_match<_Bi_iter>::string_type string_type;
+      return __rhs.compare(string_type(__lhs.data(), __lhs.size())) == 0;
+    }
 
   /**
    * @brief Tests the inequivalence of a string and a regular expression
@@ -1031,7 +1034,10 @@ 
     inline bool
     operator<(const __sub_match_string<_Bi_iter, _Ch_traits, _Ch_alloc>& __lhs,
 	      const sub_match<_Bi_iter>& __rhs)
-     { return __rhs.compare(__lhs.c_str()) > 0; }
+    {
+      typedef typename sub_match<_Bi_iter>::string_type string_type;
+      return __rhs.compare(string_type(__lhs.data(), __lhs.size())) > 0;
+    }
 
   /**
    * @brief Tests the ordering of a string and a regular expression submatch.
@@ -1080,7 +1086,10 @@ 
     inline bool
     operator==(const sub_match<_Bi_iter>& __lhs,
 	       const __sub_match_string<_Bi_iter, _Ch_traits, _Ch_alloc>& __rhs)
-    { return __lhs.compare(__rhs.c_str()) == 0; }
+    {
+      typedef typename sub_match<_Bi_iter>::string_type string_type;
+      return __lhs.compare(string_type(__rhs.data(), __rhs.size())) == 0;
+    }
 
   /**
    * @brief Tests the inequivalence of a regular expression submatch and a
@@ -1105,7 +1114,10 @@ 
     inline bool
     operator<(const sub_match<_Bi_iter>& __lhs,
 	      const __sub_match_string<_Bi_iter, _Ch_traits, _Ch_alloc>& __rhs)
-    { return __lhs.compare(__rhs.c_str()) < 0; }
+    {
+      typedef typename sub_match<_Bi_iter>::string_type string_type;
+      return __lhs.compare(string_type(__rhs.data(), __rhs.size())) < 0;
+    }
 
   /**
    * @brief Tests the ordering of a regular expression submatch and a string.
Index: ChangeLog
===================================================================
--- ChangeLog	(revision 217534)
+++ ChangeLog	(revision 217535)
@@ -1,3 +1,10 @@ 
+2014-11-13  Daniel Kruegler  <daniel.kruegler@gmail.com>
+
+	* include/bits/regex.h: Support embedded zeros in sub_match
+	comparisons (DR 2217)
+	* testsuite/28_regex/sub_match/embedded_zeros_cmp.cc:
+	Add test cases for embedded zeros
+
 2014-11-13  Tim Shen  <timshen@google.com>
 
 	PR libstdc++/63775
Index: testsuite/28_regex/sub_match/embedded_zeros_cmp.cc
===================================================================
--- testsuite/28_regex/sub_match/embedded_zeros_cmp.cc	(revision 0)
+++ testsuite/28_regex/sub_match/embedded_zeros_cmp.cc	(revision 217535)
@@ -0,0 +1,55 @@ 
+// { dg-options "-std=gnu++11" }
+
+//
+// 2014-11-13  Daniel Kruegler <daniel.kruegler@gmail.com>
+//
+// Copyright (C) 2014 Free Software Foundation, Inc.
+//
+// This file is part of the GNU ISO C++ Library.  This library is free
+// software; you can redistribute it and/or modify it under the
+// terms of the GNU General Public License as published by the
+// Free Software Foundation; either version 3, or (at your option)
+// any later version.
+//
+// This library is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License along
+// with this library; see the file COPYING3.  If not see
+// <http://www.gnu.org/licenses/>.
+
+// 28.9.2 [re.submatch.op] sub_match members, [DR 2217]
+
+#include <regex>
+#include <string>
+#include <testsuite_hooks.h>
+
+int main()
+{
+  bool test __attribute__((unused)) = true;
+
+  typedef char                          value_type;
+  typedef std::basic_string<value_type> string_type;
+  typedef std::sub_match<value_type*>   sub_match_type;
+  const string_type test_data1("abc\0d", 4);
+  value_type test_data2[] = {'a', 'b', 'c'};
+  const string_type test_data3("abc");
+
+	sub_match_type sm;
+	sm.first = std::begin(test_data2);
+	sm.second  = std::end(test_data2);
+	sm.matched = true;
+
+	VERIFY( test_data1 != sm );
+	VERIFY( sm != test_data1 );
+	VERIFY( sm < test_data1 );
+	VERIFY( !(test_data1 < sm) );
+	VERIFY( test_data1 > sm );
+
+	VERIFY( test_data3 == sm );
+	VERIFY( sm == test_data3 );
+	VERIFY( !(sm < test_data3) );
+	VERIFY( !(test_data3 < sm) );
+}