From 1ce5756f05fa57647f53d31160793e696a7fa0fa Mon Sep 17 00:00:00 2001 From: Derek Westcott Date: Mon, 20 Jun 2022 14:08:28 -0700 Subject: [PATCH] No need to serialize padding --- src/style-spec/expression/definitions/index.ts | 1 - src/style-spec/expression/definitions/literal.ts | 2 -- 2 files changed, 3 deletions(-) diff --git a/src/style-spec/expression/definitions/index.ts b/src/style-spec/expression/definitions/index.ts index 23141f577f..3767fd9069 100644 --- a/src/style-spec/expression/definitions/index.ts +++ b/src/style-spec/expression/definitions/index.ts @@ -151,7 +151,6 @@ CompoundExpression.register(expressions, { [NumberType, NumberType, NumberType, NumberType], rgba ], - // REVIEW: Need an entry for 'padding' here, to consume values produced by Literal.serialize? 'has': { type: BooleanType, overloads: [ diff --git a/src/style-spec/expression/definitions/literal.ts b/src/style-spec/expression/definitions/literal.ts index c0a3dc4261..8aee52fe22 100644 --- a/src/style-spec/expression/definitions/literal.ts +++ b/src/style-spec/expression/definitions/literal.ts @@ -59,8 +59,6 @@ class Literal implements Expression { // couldn't actually generate with a "literal" expression, // so we have to implement an equivalent serialization here return ['rgba' as unknown].concat(this.value.toArray()); - } else if (this.value instanceof Padding) { - return ['padding' as unknown].concat(this.value.toArray()); } else if (this.value instanceof Formatted) { // Same as Color return this.value.serialize();