Browse Source

[5014] Only fatal must be static

Francis Dupont 8 years ago
parent
commit
6d0171d0bc
2 changed files with 6 additions and 5 deletions
  1. 1 1
      src/bin/dhcp6/dhcp6_lexer.ll
  2. 5 4
      src/bin/dhcp6/parser_context.h

+ 1 - 1
src/bin/dhcp6/dhcp6_lexer.ll

@@ -147,7 +147,7 @@ JSONString                              \"{JSONStringCharacter}*\"
     std::string tmp(yytext+1);
     tmp.resize(tmp.size() - 1);
 
-    Parser6Context::includeFile(tmp);
+    driver.includeFile(tmp);
 }
 <DIR_EXIT>"?>" BEGIN(INITIAL);
     

+ 5 - 4
src/bin/dhcp6/parser_context.h

@@ -63,7 +63,7 @@ public:
     void scanFileEnd(FILE * f);
 
     /// @brief Divert input to an include file.
-    static void includeFile(const std::string& filename);
+    void includeFile(const std::string& filename);
 
     /// @brief Run the parser on the string specified.
     ///
@@ -81,17 +81,18 @@ public:
     ///
     /// @param loc location within the parsed file when experienced a problem.
     /// @param what string explaining the nature of the error.
-    static void error(const isc::dhcp::location& loc, const std::string& what);
+    void error(const isc::dhcp::location& loc, const std::string& what);
 
     /// @brief Error handler
     ///
     /// This is a simplified error reporting tool for possible future
     /// cases when the EvalParser is not able to handle the packet.
-    static void error(const std::string& what);
+    void error(const std::string& what);
 
     /// @brief Fatal error handler
     ///
-    /// This is for should not happen but fatal errors
+    /// This is for should not happen but fatal errors.
+    /// Used by YY_FATAL_ERROR macro so required to be static.
     static void fatal(const std::string& what);
 
  private: