Index: src/sksl/ast/SkSLASTIndexSuffix.h |
diff --git a/src/sksl/ast/SkSLASTIndexSuffix.h b/src/sksl/ast/SkSLASTIndexSuffix.h |
index 755029b0a212ffa03d744760627dd7c85907fb88..44d91fa4c4c11e4da9270bbab49130ebe9f49092 100644 |
--- a/src/sksl/ast/SkSLASTIndexSuffix.h |
+++ b/src/sksl/ast/SkSLASTIndexSuffix.h |
@@ -14,27 +14,17 @@ |
namespace SkSL { |
/** |
- * A bracketed expression, as in '[0]', indicating an array access. Empty brackets (as occur in |
- * 'float[](5, 6)' are represented with a null fExpression. |
+ * A bracketed expression, as in '[0]', indicating an array access. |
*/ |
struct ASTIndexSuffix : public ASTSuffix { |
- ASTIndexSuffix(Position position) |
- : INHERITED(position, ASTSuffix::kIndex_Kind) |
- , fExpression(nullptr) {} |
- |
ASTIndexSuffix(std::unique_ptr<ASTExpression> expression) |
- : INHERITED(expression ? expression->fPosition : Position(), ASTSuffix::kIndex_Kind) |
+ : INHERITED(expression->fPosition, ASTSuffix::kIndex_Kind) |
, fExpression(std::move(expression)) {} |
std::string description() const override { |
- if (fExpression) { |
- return "[" + fExpression->description() + "]"; |
- } else { |
- return "[]"; |
- } |
+ return "[" + fExpression->description() + "]"; |
} |
- // may be null |
std::unique_ptr<ASTExpression> fExpression; |
typedef ASTSuffix INHERITED; |