BIND 10 trac2374, updated. a5683591b24d1eaf659b3820707f4de08bb59e40 [2374] Some last cleanups

[BIND 10 source code commits]

The branch, trac2374 has been updated
       via  a5683591b24d1eaf659b3820707f4de08bb59e40 (commit)
      from  e206bfd604fd82c5747bc85e3f1e8d6de8b14908 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit a5683591b24d1eaf659b3820707f4de08bb59e40
Author: Jelte Jansen <jelte at isc.org>
Date:   Thu Nov 29 12:04:04 2012 +0100

    [2374] Some last cleanups
    
    - reduced a line that was 1 character too long
    - flipped order of check with isdigit()
    - renamed Token::NUMBER_RANGE to Token::NUMBER_OUT_OF_RANGE

-----------------------------------------------------------------------

Summary of changes:
 src/lib/dns/master_lexer.cc                      |    8 ++++----
 src/lib/dns/master_lexer.h                       |    2 +-
 src/lib/dns/tests/master_lexer_state_unittest.cc |    4 ++--
 src/lib/dns/tests/master_lexer_token_unittest.cc |    2 +-
 4 files changed, 8 insertions(+), 8 deletions(-)

-----------------------------------------------------------------------
diff --git a/src/lib/dns/master_lexer.cc b/src/lib/dns/master_lexer.cc
index ee81615..2bf0254 100644
--- a/src/lib/dns/master_lexer.cc
+++ b/src/lib/dns/master_lexer.cc
@@ -212,7 +212,7 @@ const char* const error_text[] = {
     "unexpected end of input",  // UNEXPECTED_END
     "unbalanced quotes",        // UNBALANCED_QUOTES
     "no token produced",        // NO_TOKEN_PRODUCED
-    "number out of range"       // NUMBER_RANGE
+    "number out of range"       // NUMBER_OUT_OF_RANGE
 };
 const size_t error_text_max_count = sizeof(error_text) / sizeof(error_text[0]);
 } // end unnamed namespace
@@ -379,7 +379,7 @@ State::start(MasterLexer& lexer, MasterLexer::Options options) {
                 return (NULL);
             }
             --paren_count;
-        } else if (isdigit(c) && (options & MasterLexer::NUMBER) != 0) {
+        } else if ((options & MasterLexer::NUMBER) != 0 &&isdigit(c)) {
             lexerimpl.last_was_eol_ = false;
             // this character will be handled in the number state
             lexerimpl.source_->ungetChar();
@@ -469,12 +469,12 @@ Number::handle(MasterLexer& lexer) const {
                 data.push_back('\0');
                 try {
                     const uint32_t number32 =
-                        boost::lexical_cast<uint32_t, const char*>(&data.at(0));
+                        boost::lexical_cast<uint32_t, const char*>(&data[0]);
                     token = MasterLexer::Token(number32);
                 } catch (const boost::bad_lexical_cast&) {
                     // Since we already know we have only digits,
                     // range should be the only possible problem.
-                    token = Token(Token::NUMBER_RANGE);
+                    token = Token(Token::NUMBER_OUT_OF_RANGE);
                 }
             } else {
                 token = MasterLexer::Token(&data.at(0),
diff --git a/src/lib/dns/master_lexer.h b/src/lib/dns/master_lexer.h
index 2717c87..4a861fc 100644
--- a/src/lib/dns/master_lexer.h
+++ b/src/lib/dns/master_lexer.h
@@ -292,7 +292,7 @@ public:
         UNBALANCED_QUOTES,      ///< Unbalanced quotations detected
         NO_TOKEN_PRODUCED, ///< No token was produced. This means programmer
                            /// error and should never get out of the lexer.
-        NUMBER_RANGE,    ///< Number was out of range
+        NUMBER_OUT_OF_RANGE, ///< Number was out of range
         MAX_ERROR_CODE ///< Max integer corresponding to valid error codes.
                        /// (excluding this one). Mainly for internal use.
     };
diff --git a/src/lib/dns/tests/master_lexer_state_unittest.cc b/src/lib/dns/tests/master_lexer_state_unittest.cc
index 18d2465..d8a6b66 100644
--- a/src/lib/dns/tests/master_lexer_state_unittest.cc
+++ b/src/lib/dns/tests/master_lexer_state_unittest.cc
@@ -488,12 +488,12 @@ TEST_F(MasterLexerStateTest, basicNumbers) {
 
     EXPECT_EQ(&s_number, State::start(lexer, options));
     s_number.handle(lexer);
-    EXPECT_EQ(Token::NUMBER_RANGE,
+    EXPECT_EQ(Token::NUMBER_OUT_OF_RANGE,
               s_number.getToken(lexer).getErrorCode());
 
     EXPECT_EQ(&s_number, State::start(lexer, options));
     s_number.handle(lexer);
-    EXPECT_EQ(Token::NUMBER_RANGE,
+    EXPECT_EQ(Token::NUMBER_OUT_OF_RANGE,
               s_number.getToken(lexer).getErrorCode());
 
     EXPECT_EQ(&s_number, State::start(lexer, options));
diff --git a/src/lib/dns/tests/master_lexer_token_unittest.cc b/src/lib/dns/tests/master_lexer_token_unittest.cc
index 0c13658..1f022df 100644
--- a/src/lib/dns/tests/master_lexer_token_unittest.cc
+++ b/src/lib/dns/tests/master_lexer_token_unittest.cc
@@ -146,7 +146,7 @@ TEST_F(MasterLexerTokenTest, errors) {
               MasterLexer::Token(MasterLexer::Token::NO_TOKEN_PRODUCED).
               getErrorText());
     EXPECT_EQ("number out of range",
-              MasterLexer::Token(MasterLexer::Token::NUMBER_RANGE).
+              MasterLexer::Token(MasterLexer::Token::NUMBER_OUT_OF_RANGE).
               getErrorText());
 
     // getErrorCode/Text() isn't allowed for non number types