Release 1_46_0 beta 1

[SVN r68581]

include/boost/algorithm/string/detail/find_format.hpp

@@ -76,15 +76,15 @@ namespace boost {
                 const FindResultT& FindResult )
             {   
                 if( ::boost::algorithm::detail::check_find_result(Input, FindResult) ) {
-                return ::boost::algorithm::detail::find_format_copy_impl2( 
+                    return ::boost::algorithm::detail::find_format_copy_impl2( 
-                    Output,
+                        Output,
-                    Input,
+                        Input,
-                    Formatter,
+                        Formatter,
-                    FindResult,
+                        FindResult,
-                    Formatter(FindResult) );
+                        Formatter(FindResult) );
                 } else {
                     return std::copy( ::boost::begin(Input), ::boost::end(Input), Output );
-            }
+                }
             }
 
  
@@ -137,14 +137,14 @@ namespace boost {
                 const FindResultT& FindResult)
             {
                 if( ::boost::algorithm::detail::check_find_result(Input, FindResult) ) {
-                return ::boost::algorithm::detail::find_format_copy_impl2(
+                    return ::boost::algorithm::detail::find_format_copy_impl2(
-                    Input,
+                        Input,
-                    Formatter,
+                        Formatter,
-                    FindResult,
+                        FindResult,
-                    Formatter(FindResult) );
+                        Formatter(FindResult) );
                 } else {
                     return Input;
-            }
+                }
             }
 
  // replace implementation ----------------------------------------------------//
@@ -189,12 +189,12 @@ namespace boost {
                 const FindResultT& FindResult)
             {
                 if( ::boost::algorithm::detail::check_find_result(Input, FindResult) ) {
-                ::boost::algorithm::detail::find_format_impl2(
+                    ::boost::algorithm::detail::find_format_impl2(
-                    Input,
+                        Input,
-                    Formatter,
+                        Formatter,
-                    FindResult,
+                        FindResult,
-                    Formatter(FindResult) );
+                        Formatter(FindResult) );
-            }
+                }
             }
 
         } // namespace detail

include/boost/algorithm/string/detail/find_format_all.hpp

@@ -86,16 +86,16 @@ namespace boost {
                 const FindResultT& FindResult )
             {   
                 if( ::boost::algorithm::detail::check_find_result(Input, FindResult) ) {
-                return ::boost::algorithm::detail::find_format_all_copy_impl2( 
+                    return ::boost::algorithm::detail::find_format_all_copy_impl2( 
-                    Output,
+                        Output,
-                    Input,
+                        Input,
-                    Finder,
+                        Finder,
-                    Formatter,
+                        Formatter,
-                    FindResult,
+                        FindResult,
-                    Formatter(FindResult) );
+                        Formatter(FindResult) );
                 } else {
                     return std::copy( ::boost::begin(Input), ::boost::end(Input), Output );
-            }
+                }
             }
 
  // find_format_all_copy implementation ----------------------------------------------//
@@ -161,15 +161,15 @@ namespace boost {
                 const FindResultT& FindResult)
             {
                 if( ::boost::algorithm::detail::check_find_result(Input, FindResult) ) {
-                return ::boost::algorithm::detail::find_format_all_copy_impl2(
+                    return ::boost::algorithm::detail::find_format_all_copy_impl2(
-                    Input,
+                        Input,
-                    Finder,
+                        Finder,
-                    Formatter,
+                        Formatter,
-                    FindResult,
+                        FindResult,
-                    Formatter(FindResult) );
+                        Formatter(FindResult) );
                 } else {
                     return Input;
-            }
+                }
             }
 
  // find_format_all implementation ------------------------------------------------//
@@ -257,13 +257,13 @@ namespace boost {
                 FindResultT FindResult)
             {
                 if( ::boost::algorithm::detail::check_find_result(Input, FindResult) ) {
-                ::boost::algorithm::detail::find_format_all_impl2(
+                    ::boost::algorithm::detail::find_format_all_impl2(
-                    Input,
+                        Input,
-                    Finder,
+                        Finder,
-                    Formatter,
+                        Formatter,
-                    FindResult,
+                        FindResult,
-                    Formatter(FindResult) );
+                        Formatter(FindResult) );
-            }
+                }
             }
 
         } // namespace detail

include/boost/algorithm/string/detail/find_format_store.hpp

@@ -53,7 +53,7 @@ namespace boost {
                 {
                     iterator_range<ForwardIteratorT>::operator=(FindResult);
                     if( !this->empty() ) {
-                    m_FormatResult=m_Formatter(FindResult);
+                        m_FormatResult=m_Formatter(FindResult);
                     }
                     
                     return *this;

include/boost/algorithm/string/find_iterator.hpp

@@ -259,7 +259,11 @@ namespace boost {
                 m_End(End),
                 m_bEof(false)
             {
-                increment();
+                // force the correct behavior for empty sequences and yield at least one token
+                if(Begin!=End)
+                {
+                    increment();
+                }
             }
             //! Constructor
             /*!
@@ -278,7 +282,11 @@ namespace boost {
                 m_Next=::boost::begin(lit_col);
                 m_End=::boost::end(lit_col);
 
-                increment();
+                // force the correct behavior for empty sequences and yield at least one token
+                if(m_Next!=m_End)
+                {
+                    increment();
+                }
             }
 
 

string/test/split_test.cpp

@@ -40,6 +40,7 @@ void iterator_test()
     string str1("xx-abc--xx-abb");
     string str2("Xx-abc--xX-abb-xx");
     string str3("xx");
+    string strempty("");
     const char* pch1="xx-abc--xx-abb";
     vector<string> tokens;
     vector< vector<int> > vtokens;
@@ -123,6 +124,25 @@ void iterator_test()
     BOOST_CHECK( tokens[3]==string("xx") );
     BOOST_CHECK( tokens[4]==string("abb") );
 
+    split(
+        tokens,
+        str3,
+        is_any_of(","),
+        token_compress_off);
+
+    BOOST_REQUIRE( tokens.size()==1 );
+    BOOST_CHECK( tokens[0]==string("xx") );
+
+    split(
+        tokens,
+        strempty,
+        is_punct(),
+        token_compress_off);
+
+    BOOST_REQUIRE( tokens.size()==1 );
+    BOOST_CHECK( tokens[0]==string("") );
+
+
     find_iterator<string::iterator> fiter=make_find_iterator(str1, first_finder("xx"));
     BOOST_CHECK(equals(*fiter, "xx"));
     ++fiter;