Browse Source

[2373] added a comment about if-else complexity in State::start().

JINMEI Tatuya 12 years ago
parent
commit
377f14a32c
1 changed files with 3 additions and 0 deletions
  1. 3 0
      src/lib/dns/master_lexer.cc

+ 3 - 0
src/lib/dns/master_lexer.cc

@@ -270,6 +270,9 @@ State::start(MasterLexer& lexer, MasterLexer::Options options) {
     MasterLexer::MasterLexerImpl& lexerimpl = *lexer.impl_;
     size_t& paren_count = lexerimpl.paren_count_;
 
+    // Note: the if-else in the loop is getting complicated.  When we complete
+    // #2374, revisit the organization to see if we need a fundamental
+    // refactoring.
     while (true) {
         const int c = lexerimpl.skipComment(lexerimpl.source_->getChar());
         if (c == InputSource::END_OF_STREAM) {