ashatora/openlayers
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Expressions / refactor operators to avoid linting errors

Browse Source 
Previously the utilities used the Operators dict before its definition.
This commit is contained in:
Olivier Guyot
2019-10-25 10:34:32 +02:00
parent a64f2eb720
commit 9e010631c1
2 changed files with 273 additions and 272 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
examples/filter-points-webgl.js
View File

@@ -44,7 +44,8 @@ const style = {
],
color: ['interpolate',
animRatio,
newColor, oldColor],
newColor, oldColor
],
opacity: ['-', 1.0, ['*', animRatio, 0.75]]
}
};

110
src/ol/style/expressions.js
View File

@@ -72,6 +72,22 @@ export const ValueTypes = {
NONE: 0
};
/**
* An operator declaration must contain two methods: `getReturnType` which returns a type based on
* the operator arguments, and `toGlsl` which returns a GLSL-compatible string.
* Note: both methods can process arguments recursively.
* @typedef {Object} Operator
* @property {function(Array<ExpressionValue>): ValueTypes|number} getReturnType Returns one or several types
* @property {function(ParsingContext, Array<ExpressionValue>, ValueTypes=): string} toGlsl Returns a GLSL-compatible string
* Note: takes in an optional type hint as 3rd parameter
*/
/**
* Operator declarations
* @type {Object<string, Operator>}
*/
export const Operators = {};
/**
* Returns the possible types for a given value (each type being a binary flag)
* To test a value use e.g. `getValueType(v) & ValueTypes.BOOLEAN`
@@ -258,22 +274,7 @@ function assertUniqueInferredType(args, types) {
}
}
/**
* An operator declaration must contain two methods: `getReturnType` which returns a type based on
* the operator arguments, and `toGlsl` which returns a GLSL-compatible string.
* Note: both methods can process arguments recursively.
* @typedef {Object} Operator
* @property {function(Array<ExpressionValue>): ValueTypes|number} getReturnType Returns one or several types
* @property {function(ParsingContext, Array<ExpressionValue>, ValueTypes=): string} toGlsl Returns a GLSL-compatible string
* Note: takes in an optional type hint as 3rd parameter
*/
/**
* Operator declarations
* @type {Object<string, Operator>}
*/
export const Operators = {
'get': {
Operators['get'] = {
getReturnType: function(args) {
return ValueTypes.ANY;
},
@@ -287,8 +288,8 @@ export const Operators = {
const prefix = context.inFragmentShader ? 'v_' : 'a_';
return prefix + value;
}
},
'var': {
};
Operators['var'] = {
getReturnType: function(args) {
return ValueTypes.ANY;
},
@@ -301,8 +302,8 @@ export const Operators = {
}
return `u_${value}`;
}
},
'time': {
};
Operators['time'] = {
getReturnType: function(args) {
return ValueTypes.NUMBER;
},
@@ -310,8 +311,8 @@ export const Operators = {
assertArgsCount(args, 0);
return 'u_time';
}
},
'*': {
};
Operators['*'] = {
getReturnType: function(args) {
return ValueTypes.NUMBER;
},
@@ -321,8 +322,8 @@ export const Operators = {
assertNumber(args[1]);
return `(${expressionToGlsl(context, args[0])} * ${expressionToGlsl(context, args[1])})`;
}
},
'/': {
};
Operators['/'] = {
getReturnType: function(args) {
return ValueTypes.NUMBER;
},
@@ -332,8 +333,8 @@ export const Operators = {
assertNumber(args[1]);
return `(${expressionToGlsl(context, args[0])} / ${expressionToGlsl(context, args[1])})`;
}
},
'+': {
};
Operators['+'] = {
getReturnType: function(args) {
return ValueTypes.NUMBER;
},
@@ -343,8 +344,8 @@ export const Operators = {
assertNumber(args[1]);
return `(${expressionToGlsl(context, args[0])} + ${expressionToGlsl(context, args[1])})`;
}
},
'-': {
};
Operators['-'] = {
getReturnType: function(args) {
return ValueTypes.NUMBER;
},
@@ -354,8 +355,8 @@ export const Operators = {
assertNumber(args[1]);
return `(${expressionToGlsl(context, args[0])} - ${expressionToGlsl(context, args[1])})`;
}
},
'clamp': {
};
Operators['clamp'] = {
getReturnType: function(args) {
return ValueTypes.NUMBER;
},
@@ -368,8 +369,8 @@ export const Operators = {
const max = expressionToGlsl(context, args[2]);
return `clamp(${expressionToGlsl(context, args[0])}, ${min}, ${max})`;
}
},
'stretch': {
};
Operators['stretch'] = {
getReturnType: function(args) {
return ValueTypes.NUMBER;
},
@@ -386,8 +387,8 @@ export const Operators = {
const high2 = expressionToGlsl(context, args[4]);
return `((clamp(${expressionToGlsl(context, args[0])}, ${low1}, ${high1}) - ${low1}) * ((${high2} - ${low2}) / (${high1} - ${low1})) + ${low2})`;
}
},
'mod': {
};
Operators['mod'] = {
getReturnType: function(args) {
return ValueTypes.NUMBER;
},
@@ -397,8 +398,8 @@ export const Operators = {
assertNumber(args[1]);
return `mod(${expressionToGlsl(context, args[0])}, ${expressionToGlsl(context, args[1])})`;
}
},
'pow': {
};
Operators['pow'] = {
getReturnType: function(args) {
return ValueTypes.NUMBER;
},
@@ -408,8 +409,8 @@ export const Operators = {
assertNumber(args[1]);
return `pow(${expressionToGlsl(context, args[0])}, ${expressionToGlsl(context, args[1])})`;
}
},
'>': {
};
Operators['>'] = {
getReturnType: function(args) {
return ValueTypes.BOOLEAN;
},
@@ -419,8 +420,8 @@ export const Operators = {
assertNumber(args[1]);
return `(${expressionToGlsl(context, args[0])} > ${expressionToGlsl(context, args[1])})`;
}
},
'>=': {
};
Operators['>='] = {
getReturnType: function(args) {
return ValueTypes.BOOLEAN;
},
@@ -430,8 +431,8 @@ export const Operators = {
assertNumber(args[1]);
return `(${expressionToGlsl(context, args[0])} >= ${expressionToGlsl(context, args[1])})`;
}
},
'<': {
};
Operators['<'] = {
getReturnType: function(args) {
return ValueTypes.BOOLEAN;
},
@@ -441,8 +442,8 @@ export const Operators = {
assertNumber(args[1]);
return `(${expressionToGlsl(context, args[0])} < ${expressionToGlsl(context, args[1])})`;
}
},
'<=': {
};
Operators['<='] = {
getReturnType: function(args) {
return ValueTypes.BOOLEAN;
},
@@ -452,8 +453,8 @@ export const Operators = {
assertNumber(args[1]);
return `(${expressionToGlsl(context, args[0])} <= ${expressionToGlsl(context, args[1])})`;
}
},
'==': {
};
Operators['=='] = {
getReturnType: function(args) {
return ValueTypes.BOOLEAN;
},
@@ -463,8 +464,8 @@ export const Operators = {
assertNumber(args[1]);
return `(${expressionToGlsl(context, args[0])} == ${expressionToGlsl(context, args[1])})`;
}
},
'!': {
};
Operators['!'] = {
getReturnType: function(args) {
return ValueTypes.BOOLEAN;
},
@@ -473,8 +474,8 @@ export const Operators = {
assertBoolean(args[0]);
return `(!${expressionToGlsl(context, args[0])})`;
}
},
'between': {
};
Operators['between'] = {
getReturnType: function(args) {
return ValueTypes.BOOLEAN;
},
@@ -488,8 +489,8 @@ export const Operators = {
const value = expressionToGlsl(context, args[0]);
return `(${value} >= ${min} && ${value} <= ${max})`;
}
},
'interpolate': {
};
Operators['interpolate'] = {
getReturnType: function(args) {
return ValueTypes.COLOR;
},
@@ -503,8 +504,8 @@ export const Operators = {
const end = expressionToGlsl(newContext, args[2], ValueTypes.COLOR);
return `mix(${start}, ${end}, ${expressionToGlsl(context, args[0])})`;
}
},
'match': {
};
Operators['match'] = {
getReturnType: function(args) {
let type = ValueTypes.ANY;
for (let i = 2; i < args.length; i += 2) {
@@ -531,5 +532,4 @@ export const Operators = {
}
return result;
}
}
};