initial checkin of SkSL compiler

src/sksl/ast/SkSLASTInterfaceBlock.h

+/*
+ * Copyright 2016 Google Inc.
+ *
+ * Use of this source code is governed by a BSD-style license that can be
+ * found in the LICENSE file.
+ */
+ 
+#ifndef SKSL_ASTINTERFACEBLOCK
+#define SKSL_ASTINTERFACEBLOCK
+
+#include "SkSLASTVarDeclaration.h"
+
+namespace SkSL {
+
+/**
+ * An interface block, as in:
+ *
+ * out gl_PerVertex {
+ *   layout(builtin=0) vec4 gl_Position;
+ *   layout(builtin=1) float gl_PointSize;
+ * };
+ */

[dogben, 2016/06/20 16:23:18]

nit: mention that fValueName can be empty.

+struct ASTInterfaceBlock : public ASTDeclaration {
+    ASTInterfaceBlock(Position position,
+                      ASTModifiers modifiers, 

[dogben, 2016/06/20 16:23:18]

nit: seems inconsistent that this is not std::unique_ptr.

(FWIW, I think I would generally prefer that everything is by value.)

(Same comment in other files.)

[ethannicholas, 2016/06/20 17:45:49]

Statements and expressions are both recursive structures, so I can't use by
value on those. Modifiers are one of the few cases that are both simple enough
to pass by value and that I don't have any reason to care about the identity of.

+                      std::string interfaceName, 
+                      std::string valueName, 
+                      std::vector<std::unique_ptr<ASTVarDeclaration>> declarations)
+    : INHERITED(position, kInterfaceBlock_Kind)
+    , fModifiers(modifiers)
+    , fInterfaceName(interfaceName)
+    , fValueName(valueName)
+    , fDeclarations(std::move(declarations)) {}
+
+    std::string description() const override {
+        std::string result = fModifiers.description() + fInterfaceName + " {\n";
+        for (size_t i = 0; i < fDeclarations.size(); i++) {
+            result += fDeclarations[i]->description() + "\n";
+        }
+        result += "}";
+        if (fValueName.length()) {
+            result += " " + fValueName;
+        }
+        return result + ";";
+    }
+
+    const ASTModifiers fModifiers;
+    const std::string fInterfaceName;
+    const std::string fValueName;
+    const std::vector<std::unique_ptr<ASTVarDeclaration>> fDeclarations;
+
+    typedef ASTDeclaration INHERITED;
+};
+
+} // namespace
+
+#endif