-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Rework spec function/expression taxonomy (counterpart of gl-js#6521) #11861
Conversation
Ports mapbox/mapbox-gl-js#6521: for the sake of gl-native this only requires tweak to code generation scripts.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In general, instead of updating the definition of isDataDriven
, we should replace its uses with a switch on property-type
.
}; | ||
|
||
global.isInterpolable = function(property) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't seem to be used.
@@ -319,9 +319,9 @@ global.propertyDoc = function (propertyName, property, layerType, kind) { | |||
'* Conditional expressions\n' + | |||
'* Variable assignments and references to assigned variables\n'; | |||
const inputVariable = property.name === 'heatmap-color' ? '$heatmapDensity' : '$zoomLevel'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this use expression.parameters
?
@@ -164,7 +164,7 @@ namespace mbgl { | |||
|
|||
<% if (property.name === 'heatmap-color') { -%> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this use property-type
?
👍 I left a few uses of |
This PR is mostly minor tweaks to codegen scripts to account for the spec taxonomy changes described in mapbox/mapbox-gl-js#6521, plus a few ignores needed to update the gl-js pin. (Nitpick test is failing until that PR is merged to master.)