Boost logo

Boost-Commit :

Subject: [Boost-commit] svn:boost r78480 - in trunk/boost/regex: . v4
From: john_at_[hidden]
Date: 2012-05-15 12:09:13


Author: johnmaddock
Date: 2012-05-15 12:09:12 EDT (Tue, 15 May 2012)
New Revision: 78480
URL: http://svn.boost.org/trac/boost/changeset/78480

Log:
Fix some more GCC warnings.
Fix previous warning suppression which broke tests with GCC.
Refs #6863.
Text files modified:
   trunk/boost/regex/concepts.hpp | 2 +
   trunk/boost/regex/v4/basic_regex_creator.hpp | 42 ++++++++++++++++++++--------------------
   trunk/boost/regex/v4/basic_regex_parser.hpp | 8 +++---
   trunk/boost/regex/v4/perl_matcher_non_recursive.hpp | 8 +++---
   4 files changed, 31 insertions(+), 29 deletions(-)

Modified: trunk/boost/regex/concepts.hpp
==============================================================================
--- trunk/boost/regex/concepts.hpp (original)
+++ trunk/boost/regex/concepts.hpp 2012-05-15 12:09:12 EDT (Tue, 15 May 2012)
@@ -63,6 +63,8 @@
    // conversion to integral type:
    operator long()const;
 };
+inline long hash_value(char_architype val)
+{ return val; }
 //
 // char_architype can not be used with basic_string:
 //

Modified: trunk/boost/regex/v4/basic_regex_creator.hpp
==============================================================================
--- trunk/boost/regex/v4/basic_regex_creator.hpp (original)
+++ trunk/boost/regex/v4/basic_regex_creator.hpp 2012-05-15 12:09:12 EDT (Tue, 15 May 2012)
@@ -66,7 +66,7 @@
 public:
    typedef digraph<charT> digraph_type;
    typedef typename traits::string_type string_type;
- typedef typename traits::char_class_type mask_type;
+ typedef typename traits::char_class_type m_type;
 
    basic_char_set()
    {
@@ -100,12 +100,12 @@
       }
       m_empty = false;
    }
- void add_class(mask_type m)
+ void add_class(m_type m)
    {
       m_classes |= m;
       m_empty = false;
    }
- void add_negated_class(mask_type m)
+ void add_negated_class(m_type m)
    {
       m_negated_classes |= m;
       m_empty = false;
@@ -162,11 +162,11 @@
    {
       return m_equivalents.end();
    }
- mask_type classes()const
+ m_type classes()const
    {
       return m_classes;
    }
- mask_type negated_classes()const
+ m_type negated_classes()const
    {
       return m_negated_classes;
    }
@@ -179,8 +179,8 @@
    std::vector<digraph_type> m_ranges; // a list of end points of our ranges
    bool m_negate; // true if the set is to be negated
    bool m_has_digraphs; // true if we have digraphs present
- mask_type m_classes; // character classes to match
- mask_type m_negated_classes; // negated character classes to match
+ m_type m_classes; // character classes to match
+ m_type m_negated_classes; // negated character classes to match
    bool m_empty; // whether we've added anything yet
    std::vector<digraph_type> m_equivalents; // a list of equivalence classes
 };
@@ -367,9 +367,9 @@
 {
    typedef typename traits::string_type string_type;
    typedef typename basic_char_set<charT, traits>::list_iterator item_iterator;
- typedef typename traits::char_class_type mask_type;
+ typedef typename traits::char_class_type m_type;
    
- re_set_long<mask_type>* result = static_cast<re_set_long<mask_type>*>(append_state(syntax_element_long_set, sizeof(re_set_long<mask_type>)));
+ re_set_long<m_type>* result = static_cast<re_set_long<m_type>*>(append_state(syntax_element_long_set, sizeof(re_set_long<m_type>)));
    //
    // fill in the basics:
    //
@@ -513,7 +513,7 @@
    //
    // finally reset the address of our last state:
    //
- m_last_state = result = static_cast<re_set_long<mask_type>*>(getaddress(offset));
+ m_last_state = result = static_cast<re_set_long<m_type>*>(getaddress(offset));
    return result;
 }
 
@@ -593,7 +593,7 @@
       }
       else
       {
- if(char_less<charT>(c2, c1))
+ if(char_less(c2, c1))
          {
             // Oops error:
             return 0;
@@ -605,8 +605,8 @@
    //
    // and now the classes:
    //
- typedef typename traits::char_class_type mask_type;
- mask_type m = char_set.classes();
+ typedef typename traits::char_class_type m_type;
+ m_type m = char_set.classes();
    if(flags() & regbase::icase)
    {
       // adjust m as needed:
@@ -977,7 +977,7 @@
 template <class charT, class traits>
 int basic_regex_creator<charT, traits>::calculate_backstep(re_syntax_base* state)
 {
- typedef typename traits::char_class_type mask_type;
+ typedef typename traits::char_class_type m_type;
    int result = 0;
    while(state)
    {
@@ -1033,7 +1033,7 @@
             else if(state->type == syntax_element_long_set_rep)
             {
                BOOST_ASSERT(rep->next.p->type == syntax_element_long_set);
- if(static_cast<re_set_long<mask_type>*>(rep->next.p)->singleton == 0)
+ if(static_cast<re_set_long<m_type>*>(rep->next.p)->singleton == 0)
                   return -1;
                if(rep->max != rep->min)
                   return -1;
@@ -1044,7 +1044,7 @@
          }
          return -1;
       case syntax_element_long_set:
- if(static_cast<re_set_long<mask_type>*>(state)->singleton == 0)
+ if(static_cast<re_set_long<m_type>*>(state)->singleton == 0)
             return -1;
          result += 1;
          break;
@@ -1214,14 +1214,14 @@
       case syntax_element_long_set:
          if(l_map)
          {
- typedef typename traits::char_class_type mask_type;
- if(static_cast<re_set_long<mask_type>*>(state)->singleton)
+ typedef typename traits::char_class_type m_type;
+ if(static_cast<re_set_long<m_type>*>(state)->singleton)
             {
                l_map[0] |= mask_init;
                for(unsigned int i = 0; i < (1u << CHAR_BIT); ++i)
                {
                   charT c = static_cast<charT>(i);
- if(&c != re_is_set_member(&c, &c + 1, static_cast<re_set_long<mask_type>*>(state), *m_pdata, l_icase))
+ if(&c != re_is_set_member(&c, &c + 1, static_cast<re_set_long<m_type>*>(state), *m_pdata, l_icase))
                      l_map[i] |= mask;
                }
             }
@@ -1464,7 +1464,7 @@
 template <class charT, class traits>
 syntax_element_type basic_regex_creator<charT, traits>::get_repeat_type(re_syntax_base* state)
 {
- typedef typename traits::char_class_type mask_type;
+ typedef typename traits::char_class_type m_type;
    if(state->type == syntax_element_rep)
    {
       // check to see if we are repeating a single state:
@@ -1479,7 +1479,7 @@
          case re_detail::syntax_element_set:
             return re_detail::syntax_element_short_set_rep;
          case re_detail::syntax_element_long_set:
- if(static_cast<re_detail::re_set_long<mask_type>*>(state->next.p)->singleton)
+ if(static_cast<re_detail::re_set_long<m_type>*>(state->next.p)->singleton)
                return re_detail::syntax_element_long_set_rep;
             break;
          default:

Modified: trunk/boost/regex/v4/basic_regex_parser.hpp
==============================================================================
--- trunk/boost/regex/v4/basic_regex_parser.hpp (original)
+++ trunk/boost/regex/v4/basic_regex_parser.hpp 2012-05-15 12:09:12 EDT (Tue, 15 May 2012)
@@ -669,8 +669,8 @@
    case regex_constants::escape_type_class:
       {
 escape_type_class_jump:
- typedef typename traits::char_class_type mask_type;
- mask_type m = this->m_traits.lookup_classname(m_position, m_position+1);
+ typedef typename traits::char_class_type m_type;
+ m_type m = this->m_traits.lookup_classname(m_position, m_position+1);
          if(m != 0)
          {
             basic_char_set<charT, traits> char_set;
@@ -1383,8 +1383,8 @@
          ++name_first;
          negated = true;
       }
- typedef typename traits::char_class_type mask_type;
- mask_type m = this->m_traits.lookup_classname(name_first, name_last);
+ typedef typename traits::char_class_type m_type;
+ m_type m = this->m_traits.lookup_classname(name_first, name_last);
       if(m == 0)
       {
          if(char_set.empty() && (name_last - name_first == 1))

Modified: trunk/boost/regex/v4/perl_matcher_non_recursive.hpp
==============================================================================
--- trunk/boost/regex/v4/perl_matcher_non_recursive.hpp (original)
+++ trunk/boost/regex/v4/perl_matcher_non_recursive.hpp 2012-05-15 12:09:12 EDT (Tue, 15 May 2012)
@@ -828,9 +828,9 @@
 #ifdef __BORLANDC__
 #pragma option push -w-8008 -w-8066 -w-8004
 #endif
- typedef typename traits::char_class_type mask_type;
+ typedef typename traits::char_class_type m_type;
    const re_repeat* rep = static_cast<const re_repeat*>(pstate);
- const re_set_long<mask_type>* set = static_cast<const re_set_long<mask_type>*>(pstate->next.p);
+ const re_set_long<m_type>* set = static_cast<const re_set_long<m_type>*>(pstate->next.p);
    std::size_t count = 0;
    //
    // start by working out how much we can skip:
@@ -1434,7 +1434,7 @@
 template <class BidiIterator, class Allocator, class traits>
 bool perl_matcher<BidiIterator, Allocator, traits>::unwind_long_set_repeat(bool r)
 {
- typedef typename traits::char_class_type mask_type;
+ typedef typename traits::char_class_type m_type;
    saved_single_repeat<BidiIterator>* pmp = static_cast<saved_single_repeat<BidiIterator>*>(m_backup_state);
 
    // if we have a match, just discard this state:
@@ -1447,7 +1447,7 @@
    const re_repeat* rep = pmp->rep;
    std::size_t count = pmp->count;
    pstate = rep->next.p;
- const re_set_long<mask_type>* set = static_cast<const re_set_long<mask_type>*>(pstate);
+ const re_set_long<m_type>* set = static_cast<const re_set_long<m_type>*>(pstate);
    position = pmp->last_position;
 
    BOOST_ASSERT(rep->type == syntax_element_long_set_rep);


Boost-Commit list run by bdawes at acm.org, david.abrahams at rcn.com, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk