[Boost-commit] svn:boost r78839 - trunk/boost/algorithm

Subject: [Boost-commit] svn:boost r78839 - trunk/boost/algorithm
From: marshall_at_[hidden]
Date: 2012-06-06 16:43:23

Author: marshall
Date: 2012-06-06 16:43:22 EDT (Wed, 06 Jun 2012)
New Revision: 78839
URL: http://svn.boost.org/trac/boost/changeset/78839

Log:
Removed a bit of undefined behavior (noted by STL @ microsoft)
Text files modified: 
   trunk/boost/algorithm/hex.hpp |     6 ++----                                  
   1 files changed, 2 insertions(+), 4 deletions(-)

Modified: trunk/boost/algorithm/hex.hpp
==============================================================================
--- trunk/boost/algorithm/hex.hpp	(original)
+++ trunk/boost/algorithm/hex.hpp	2012-06-06 16:43:22 EDT (Wed, 06 Jun 2012)
@@ -73,7 +73,7 @@
         if ( c >= '0' && c <= '9' ) return c - '0';
         if ( c >= 'A' && c <= 'F' ) return c - 'A' + 10;
         if ( c >= 'a' && c <= 'f' ) return c - 'a' + 10;
-        BOOST_THROW_EXCEPTION (non_hex_input() << boost::algorithm::bad_char (c));
+        BOOST_THROW_EXCEPTION (non_hex_input() << bad_char (c));
         return 0;   // keep dumb compilers happy
         }
     
@@ -223,10 +223,8 @@
 //  If we run into the terminator while decoding, we will throw a
 //      malformed input exception. It would be nicer to throw a 'Not enough input'
 //      exception - but how much extra work would that require?
-//  I just make up an "end iterator" which we will never get to - 
-//      two Ts per byte of the output type.
     while ( *ptr )
-        out = detail::decode_one ( ptr, ptr + 2 * sizeof(OutputType), out );
+        out = detail::decode_one ( ptr, (const T *) NULL, out );
     return out;
     }