Upgrade generated parse_line() to accept constant SBuf

src/ConfigParser.cc

@@ -16,13 +16,14 @@
 #include "fatal.h"
 #include "globals.h"
 #include "neighbors.h"
+#include "parser/Tokenizer.h"
 #include "sbuf/Stream.h"
 
 bool ConfigParser::RecognizeQuotedValues = true;
 bool ConfigParser::StrictMode = true;
 std::stack<ConfigParser::CfgFile *> ConfigParser::CfgFiles;
 ConfigParser::TokenType ConfigParser::LastTokenType = ConfigParser::SimpleToken;
-const char *ConfigParser::CfgLine = nullptr;
+SBuf ConfigParser::CfgLine;
 const char *ConfigParser::CfgPos = nullptr;
 std::queue<char *> ConfigParser::CfgLineTokens_;
 bool ConfigParser::AllowMacros_ = false;
@@ -201,10 +202,10 @@ ConfigParser::UnQuote(const char *token, const char **next)
 }
 
 void
-ConfigParser::SetCfgLine(char *line)
+ConfigParser::SetCfgLine(const SBuf &line)
 {
     CfgLine = line;
-    CfgPos = line;
+    CfgPos = CfgLine.c_str();
     while (!CfgLineTokens_.empty()) {
         char *token = CfgLineTokens_.front();
         CfgLineTokens_.pop();
@@ -561,6 +562,20 @@ ConfigParser::rejectDuplicateDirective()
     throw TextException("duplicate configuration directive", Here());
 }
 
+SBuf
+ConfigParser::openDirective(const SBuf &line)
+{
+    Parser::Tokenizer tk(line);
+    SBuf directiveName; // TODO: Upgrade cfg_directive to a ConfigParser member (with SBuf type) and set it here.
+    static const auto &spaceChars = CharacterSet::WSP;
+    static const auto directiveChars = spaceChars.complement("squid.conf directive name");
+    const auto found = tk.prefix(directiveName, directiveChars);
+    Assure(found); // our callers are expected to fully handle non-directive lines
+    tk.skipAll(spaceChars);
+    SetCfgLine(tk.remaining()); // may be empty
+    return directiveName;
+}
+
 void
 ConfigParser::closeDirective()
 {

src/ConfigParser.h

@@ -54,6 +54,11 @@ class ConfigParser
 
     void destruct();
 
+    /// starts parsing the current configuration directive
+    /// \returns directive name
+    /// \sa closeDirective()
+    SBuf openDirective(const SBuf &line);
+
     /// stops parsing the current configuration directive
     void closeDirective();
 
@@ -137,8 +142,8 @@ class ConfigParser
      */
     static char *PeekAtToken();
 
-    /// Set the configuration file line to parse.
-    static void SetCfgLine(char *line);
+    /// Set current directive parameters (i.e. characters after the directive name).
+    static void SetCfgLine(const SBuf &);
 
     /// Allow %macros inside quoted strings
     static void EnableMacros() {AllowMacros_ = true;}
@@ -221,8 +226,12 @@ class ConfigParser
     static char *NextElement(TokenType &type);
     static std::stack<CfgFile *> CfgFiles; ///< The stack of open cfg files
     static TokenType LastTokenType; ///< The type of last parsed element
-    static const char *CfgLine; ///< The current line to parse
-    static const char *CfgPos; ///< Pointer to the next element in cfgLine string
+
+    static SBuf CfgLine; ///< Directive parameters being parsed; \sa SetCfgLine()
+
+    // TODO: Replace with a Tokenizer or a similar iterative parsing class
+    static const char *CfgPos; ///< Pointer to the next element in CfgLine string
+
     static std::queue<char *> CfgLineTokens_; ///< Store the list of tokens for current configuration line
     static bool AllowMacros_;
     static bool ParseQuotedOrToEol_; ///< The next tokens will be handled as quoted or to_eol token

src/Makefile.am

@@ -2748,7 +2748,6 @@ tests_testConfigParser_SOURCES = \
 	tests/testConfigParser.cc
 nodist_tests_testConfigParser_SOURCES = \
 	ConfigParser.cc \
-	tests/stub_SBuf.cc \
 	String.cc \
 	tests/stub_acl.cc \
 	tests/stub_cache_cf.cc \
@@ -2757,6 +2756,8 @@ nodist_tests_testConfigParser_SOURCES = \
 	tests/stub_libmem.cc \
 	tests/stub_neighbors.cc
 tests_testConfigParser_LDADD = \
+	parser/libparser.la \
+	sbuf/libsbuf.la \
 	base/libbase.la \
 	$(LIBCPPUNIT_LIBS) \
 	$(COMPAT_LIB) \