Get rid of variable struct

7 files changed, 60 insertions, 158 deletions

diff --git a/bbot/build-config b/bbot/build-config
index 0a3ecb3..0f92eb8 100644
--- a/bbot/build-config
+++ b/bbot/build-config
@@ -16,8 +16,6 @@
 
 #include <bbot/export>
 
-#include <bbot/variable>
-
 namespace bbot
 {
   // Build configuration matching specific machine names. Used by bbot
@@ -30,7 +28,7 @@ namespace bbot
 
     butl::optional<butl::target_triplet> target;
 
-    variables vars;
+    std::vector<std::string> vars;
   };
 
   using build_configs = std::vector<build_config>;
diff --git a/bbot/build-config.cxx b/bbot/build-config.cxx
index 226ead3..0ee00e9 100644
--- a/bbot/build-config.cxx
+++ b/bbot/build-config.cxx
@@ -13,6 +13,8 @@
 #include <butl/fdstream>
 #include <butl/tab-parser>
 
+#include <bbot/manifest> // task_manifest::check_config()
+
 using namespace std;
 using namespace butl;
 
@@ -34,17 +36,17 @@ namespace bbot
       // then we refer to the end-of-line column (presumably reporting a missed
       // field).
       //
-      auto bad_line = [&name, &tl, &i, n] (const string& d, size_t offset = 0)
+      auto bad_line = [&name, &tl, &i, n] (const string& d)
       {
         // Offset beyond the end-of-line is meaningless.
         //
-        assert (i < n || (i == n && offset == 0));
+        assert (i <= n);
 
         throw tab_parsing (name,
                            tl.line,
                            i == n
                            ? tl.end_column
-                           : tl[i].column + offset,
+                           : tl[i].column,
                            d);
       };
 
@@ -93,12 +95,14 @@ namespace bbot
       try
       {
         for (; i < n; ++i)
-          config.vars.emplace_back (variable (move (tl[i].value)));
+        {
+          task_manifest::check_config (tl[i].value);
+          config.vars.emplace_back (move (tl[i].value));
+        }
       }
-      catch (const invalid_variable& e)
+      catch (const invalid_argument& e)
       {
-        bad_line (e.what (), e.pos); // Note that tl[i].value is moved from,
-                                     // but happily we don't use it
+        bad_line (e.what ());
       }
 
       // Save the configuration.
diff --git a/bbot/buildfile b/bbot/buildfile
index 1bb608f..27bde1a 100644
--- a/bbot/buildfile
+++ b/bbot/buildfile
@@ -8,7 +8,6 @@ lib{bbot}:                        \
 {hxx         cxx}{ build-config } \
 {hxx            }{ export       } \
 {hxx         cxx}{ manifest     } \
-{hxx         cxx}{ variable     } \
 {hxx            }{ version      } \
                    $int_libs
 
diff --git a/bbot/manifest b/bbot/manifest
index 3022535..087fd25 100644
--- a/bbot/manifest
+++ b/bbot/manifest
@@ -17,8 +17,6 @@
 
 #include <bbot/export>
 
-#include <bbot/variable>
-
 namespace bbot
 {
   using strings = std::vector<std::string>;
@@ -105,8 +103,12 @@ namespace bbot
 
     // Build system configuration variables (in addition to build environment
     // configuration variables).
+    // Note: could be quoted.
     //
-    variables config;
+    strings config;
+
+    strings
+    unquoted_config ();
 
     task_manifest (std::string nm,
                    bpkg::version vr,
@@ -114,7 +116,7 @@ namespace bbot
                    strings tr,
                    std::string mn,
                    butl::optional<butl::target_triplet> tg,
-                   variables cf)
+                   strings cf)
         : name (std::move (nm)),
           version (std::move (vr)),
           repository (std::move (rl)),
@@ -132,6 +134,13 @@ namespace bbot
 
     void
     serialize (butl::manifest_serializer&) const;
+
+    // Check that a string has the name=value format. The name must not
+    // contain spaces. Throw invalid_argument if the string doesn't conform to
+    // these rules.
+    //
+    static void
+    check_config (const std::string&);
   };
 
   class LIBBBOT_EXPORT task_response_manifest
diff --git a/bbot/manifest.cxx b/bbot/manifest.cxx
index 0f7a053..1d164db 100644
--- a/bbot/manifest.cxx
+++ b/bbot/manifest.cxx
@@ -16,6 +16,7 @@
 
 #include <butl/utility>             // digit()
 #include <butl/tab-parser>
+#include <butl/string-parser>
 #include <butl/manifest-parser>
 #include <butl/manifest-serializer>
 
@@ -442,14 +443,16 @@ namespace bbot
             {
               try
               {
-                config.emplace_back (variable (move (tf.value)));
+                check_config (tf.value);
               }
-              catch (const invalid_variable& e)
+              catch (const invalid_argument& e)
               {
                 bad_value (string ("invalid task configuration: ") + e.what (),
-                           tf.column - 1 + e.pos,
+                           tf.column - 1,
                            tl.line - 1);
               }
+
+              config.emplace_back (move (tf.value));
             }
           }
         }
@@ -520,6 +523,35 @@ namespace bbot
     s.next ("", ""); // End of manifest.
   }
 
+  strings task_manifest::
+  unquoted_config ()
+  {
+    return string_parser::unquote (config);
+  }
+
+  void task_manifest::
+  check_config (const string& s)
+  {
+    auto i (s.begin ());
+    auto e (s.end ());
+
+    // Iterate until the variable name end and check that it contains no
+    // whitespaces.
+    //
+    for (; i != e; ++i)
+    {
+      char c (*i);
+
+      if (c == ' ' || c == '\t') // Whitespace in name.
+        throw invalid_argument ("expected variable assignment");
+      else if (c == '=')
+        break;
+    }
+
+    if (i == e)
+      throw invalid_argument ("no variable value");
+  }
+
   // task_response_manifest
   //
   task_response_manifest::
diff --git a/bbot/variable b/bbot/variable
deleted file mode 100644
index 11e84ec..0000000
--- a/bbot/variable
+++ /dev/null
@@ -1,46 +0,0 @@
-// file      : bbot/variable -*- C++ -*-
-// copyright : Copyright (c) 2014-2017 Code Synthesis Ltd
-// license   : MIT; see accompanying LICENSE file
-
-#ifndef BBOT_VARIABLE
-#define BBOT_VARIABLE
-
-#include <string>
-#include <vector>
-#include <cstdint>   // uint64_t
-#include <stdexcept> // invalid_argument
-
-#include <bbot/export>
-
-namespace bbot
-{
-  class LIBBBOT_EXPORT invalid_variable: public std::invalid_argument
-  {
-  public:
-    invalid_variable (std::uint64_t p, const std::string& d)
-        : invalid_argument (d), pos (p) {}
-
-    std::uint64_t pos; // Zero-based.
-  };
-
-  // String in the name=value format. Can contain single or double quoted
-  // substrings. No escaping is supported. The name must not contain spaces.
-  // Throw variable_error if the string doesn't conform to these rules.
-  //
-  struct LIBBBOT_EXPORT variable: std::string
-  {
-    variable (std::string);
-
-    // Remove a single level of quotes. Don't validate the format or the
-    // correctness of the quotation. Note that the variable can potentially be
-    // modified through the std::string interface in a way that breaks
-    // format/quoting.
-    //
-    std::string
-    unquoted () const;
-  };
-
-  using variables = std::vector<variable>;
-}
-
-#endif // BBOT_VARIABLE
diff --git a/bbot/variable.cxx b/bbot/variable.cxx
deleted file mode 100644
index a25b007..0000000
--- a/bbot/variable.cxx
+++ /dev/null
@@ -1,94 +0,0 @@
-// file      : bbot/variable.cxx -*- C++ -*-
-// copyright : Copyright (c) 2014-2017 Code Synthesis Ltd
-// license   : MIT; see accompanying LICENSE file
-
-#include <bbot/variable>
-
-#include <utility> // move()
-
-using namespace std;
-
-namespace bbot
-{
-  variable::
-  variable (string v): string (move (v))
-  {
-    // Scan the string untill the end to check that the quoting is terminated.
-    // We will also make sure that the name doesn't contain spaces and the
-    // value is provided.
-    //
-    char quoting ('\0'); // Current quoting mode, can be used as bool.
-    bool name (true);    // True while we are parsing the variable name.
-
-    auto b (cbegin ());
-    auto i (b);
-
-    auto bad_variable = [&b, &i] (const string& d)
-    {
-      throw invalid_variable (i - b, d);
-    };
-
-    for (auto e (cend ()); i != e; ++i)
-    {
-      char c (*i);
-
-      if (!quoting)
-      {
-        if (c == '"' || c == '\'') // Begin of quoted string,
-        {
-          quoting = c;
-          continue;
-        }
-      }
-      else if (c == quoting)       // End of quoted string,
-      {
-        quoting = '\0';
-        continue;
-      }
-
-      if (name)
-      {
-        if (c == ' ' || c == '\t')
-          bad_variable ("expected variable assignment");
-        else if (c == '=')
-          name = false;
-      }
-    }
-
-    if (quoting)
-      bad_variable ("unterminated quoted string");
-
-    if (name)
-      bad_variable ("no variable value");
-  }
-
-  string variable::
-  unquoted () const
-  {
-    string r;
-    char quoting ('\0');  // Current quoting mode, can be used as bool.
-
-    for (auto i (cbegin ()), e (cend ()); i != e; ++i)
-    {
-      char c (*i);
-
-      if (!quoting)
-      {
-        if (c == '"' || c == '\'') // Begin of quoted string.
-        {
-          quoting = c;
-          continue;
-        }
-      }
-      else if (c == quoting)       // End of quoted string.
-      {
-        quoting = '\0';
-        continue;
-      }
-
-      r += c;
-    }
-
-    return r;
-  }
-}