Boost C++ Libraries: Ticket #698: The case insensitive modifier doesn't work when followed by a character class

John Maddock — Tue, 01 Aug 2006 09:37:52 GMT

Logged In: YES
user_id=14804
I can't reproduce this, the test program I'm using is below,
can you check and see if this reproduces the issue for you?
 BTW I'm testing with the latest Boost-cvs, but the only
patches I'm aware of making are to non-greedy repeats which
shouldn't have any effect here.
Re equivalence classes: there is no portable way to make
this work unfortunately, it requires that the regex engine
is able to decode the collation string produced by the
locale to extract the primary equivalence class.  The "kind"
of sort key used by the platform is determined in a fairly
heuristic way in find_sort_syntax() in
boost/regex/v4/primary_transform.hpp, and the actually sort
key is produced in cpp_regex_traits::primary_transform().
You may - with a bit of debugging - be able to find out
what's going wrong (I don't have access to a mac BTW).  The
most important thing would be to find out what kind of sort
keys are returned by std::collate<>::transform.
Test program follows, John Maddock.
#include <boost/regex.hpp>
#include <iostream>
int main(int,char**)
{
   boost::regex e("(?i)[bc]");
   std::string s("ABC abc aCb");
   boost::sregex_iterator i(s.begin(), s.end(), e), j;
   while(i != j)
   {
      std::cout << *i << std::endl;
      ++i;
   }
}

status changed

sf-robot — Wed, 16 Aug 2006 02:20:23 GMT

status assigned → closed

Logged In: YES
user_id=1312539
This Tracker item was closed automatically by the system. It was
previously set to a Pending status, and the original submitter
did not respond within 14 days (the time period specified by
the administrator of this Tracker).

severity set

Thu, 25 Aug 2011 10:37:47 GMT

severity → Problem

This bug still exists as described in the original post. I've tested with Boost 1.45 but I believe it's still there in 1.47.

The regular expression "(?i)[dh]og" should match on both "HOG" and "dog" but in fact only matches on "dog". Please see the attached sample code boostbug698.cpp.

I've fixed the bug as follows: at boost/regex/v4/basic_regex_creator.hpp line 1216 change m_icase to l_icase.

< if(&c != re_is_set_member(&c, &c + 1, static_cast<re_set_long<mask_type>*>(state), *m_pdata, m_icase))
> if(&c != re_is_set_member(&c, &c + 1, static_cast<re_set_long<mask_type>*>(state), *m_pdata, l_icase))

attachment set

Thu, 25 Aug 2011 10:39:06 GMT

attachment → boostbug698.cpp

Sample code demonstrating this bug (compiles with MSVC9)

status changed; resolution deleted

Thu, 25 Aug 2011 10:40:47 GMT

status closed → reopened
resolution None

version, summary changed

Thu, 25 Aug 2011 10:41:54 GMT

version None → Boost 1.45.0
summary The case insensitive modifier doesn't work → The case insensitive modifier doesn't work when followed by a character class

description changed

John Maddock — Mon, 10 Oct 2011 16:42:41 GMT

description modified (diff)

Confirmed.

status changed; resolution set

John Maddock — Mon, 10 Oct 2011 16:55:29 GMT

status reopened → closed
resolution → fixed

(In [74898]) Fix case change bug. Fixes #698.