[2369] Rename exception to UngetBeforeBeginning

diff --git a/src/lib/dns/master_lexer_inputsource.cc b/src/lib/dns/master_lexer_inputsource.cc
index 048564c59a3daf5c6e23a3a2b6c31cd2dfd9f36e..d3570d5873f2bf294d5c7c79abed6becc1209037 100644 (file)
--- a/src/lib/dns/master_lexer_inputsource.cc
+++ b/src/lib/dns/master_lexer_inputsource.cc
@@ -90,7 +90,8 @@ InputSource::ungetChar() {
     if (at_eof_) {
         at_eof_ = false;
     } else if (buffer_pos_ == 0) {
-        isc_throw(UngetError, "Cannot skip before the start of buffer");
+        isc_throw(UngetBeforeBeginning,
+                  "Cannot skip before the start of buffer");
     } else {
         buffer_pos_--;
         if (buffer_[buffer_pos_] == '\n') {

diff --git a/src/lib/dns/master_lexer_inputsource.h b/src/lib/dns/master_lexer_inputsource.h
index ba2f77dab42dba54a70bce060a5d7823a905ae78..642d10e7835f6256ac3f9c7b262a1f1b81770228 100644 (file)
--- a/src/lib/dns/master_lexer_inputsource.h
+++ b/src/lib/dns/master_lexer_inputsource.h
@@ -73,8 +73,8 @@ public:
 
     /// \brief Exception thrown when ungetChar() is made to go before
     /// the start of buffer.
-    struct UngetError : public OutOfRange {
-        UngetError(const char* file, size_t line, const char* what) :
+    struct UngetBeforeBeginning : public OutOfRange {
+        UngetBeforeBeginning(const char* file, size_t line, const char* what) :
             OutOfRange(file, line, what)
         {}
     };
@@ -85,6 +85,10 @@ public:
 
     /// \brief Skips backward a single character in the input
     /// source. The last-read character is unget.
+    ///
+    /// \throws UngetBeforeBeginning if we go backwards past the start
+    /// of reading, or backwards past the last time compact() was
+    /// called.
     void ungetChar();
 
     /// Forgets everything read so far, and skips back to the position

diff --git a/src/lib/dns/tests/master_lexer_inputsource_unittest.cc b/src/lib/dns/tests/master_lexer_inputsource_unittest.cc
index 979e5611a829d9d660b9278250019fce2290769f..27426b3b6bdc119fedd392465a5c1339a5fdc23d 100644 (file)
--- a/src/lib/dns/tests/master_lexer_inputsource_unittest.cc
+++ b/src/lib/dns/tests/master_lexer_inputsource_unittest.cc
@@ -102,7 +102,7 @@ TEST_F(InputSourceTest, getAndUngetChar) {
     }
 
     // Skipping past the start of buffer should throw.
-    EXPECT_THROW(source_.ungetChar(), InputSource::UngetError);
+    EXPECT_THROW(source_.ungetChar(), InputSource::UngetBeforeBeginning);
 }
 
 // ungetAll() should skip back to the place where the InputSource
@@ -128,7 +128,7 @@ TEST_F(InputSourceTest, compact) {
     source_.compact();
 
     // Ungetting here must throw.
-    EXPECT_THROW(source_.ungetChar(), InputSource::UngetError);
+    EXPECT_THROW(source_.ungetChar(), InputSource::UngetBeforeBeginning);
 
     for (size_t i = 0; i < str_length_; i++) {
         EXPECT_EQ(str_[i], source_.getChar());
@@ -156,7 +156,7 @@ TEST_F(InputSourceTest, compact) {
     source_.ungetChar();
 
     // Ungetting here must throw.
-    EXPECT_THROW(source_.ungetChar(), InputSource::UngetError);
+    EXPECT_THROW(source_.ungetChar(), InputSource::UngetBeforeBeginning);
 
     EXPECT_EQ(-1, source_.getChar());
     EXPECT_TRUE(source_.atEOF());
@@ -181,7 +181,7 @@ TEST_F(InputSourceTest, compactDuring) {
     source_.compact();
 
     // Ungetting here must throw.
-    EXPECT_THROW(source_.ungetChar(), InputSource::UngetError);
+    EXPECT_THROW(source_.ungetChar(), InputSource::UngetBeforeBeginning);
 
     for (size_t i = 13; i < str_length_; i++) {
         EXPECT_EQ(str_[i], source_.getChar());
@@ -201,7 +201,7 @@ TEST_F(InputSourceTest, compactDuring) {
     source_.ungetAll();
 
     // Ungetting here must throw.
-    EXPECT_THROW(source_.ungetChar(), InputSource::UngetError);
+    EXPECT_THROW(source_.ungetChar(), InputSource::UngetBeforeBeginning);
 
     for (size_t i = 13; i < str_length_; i++) {
         EXPECT_EQ(str_[i], source_.getChar());
@@ -263,7 +263,7 @@ TEST_F(InputSourceTest, lines) {
                          ((line - 1) == source_.getCurrentLine())));
             line = source_.getCurrentLine();
         }
-    }, InputSource::UngetError);
+    }, InputSource::UngetBeforeBeginning);
 
     // Now we are back to where we started.
     EXPECT_EQ(1, source_.getCurrentLine());