diff options
author | tobrun <tobrun.van.nuland@gmail.com> | 2018-05-29 13:10:16 +0200 |
---|---|---|
committer | Tobrun <tobrun@mapbox.com> | 2018-05-29 15:53:38 +0200 |
commit | 83ebd5f6ff56fddd4a2923f6c082977476cddd99 (patch) | |
tree | b56ab08a2490f100efc91ebae8d97c987dd0f775 /platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox | |
parent | 2e2d5bf03c23b2ece3d977532c589275681973be (diff) | |
download | qtlocation-mapboxgl-83ebd5f6ff56fddd4a2923f6c082977476cddd99.tar.gz |
[android] - fix literal wrapping in comparission expressions
Diffstat (limited to 'platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox')
-rw-r--r-- | platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/expressions/Expression.java | 29 |
1 files changed, 15 insertions, 14 deletions
diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/expressions/Expression.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/expressions/Expression.java index c8f848f4d7..eb9150ecbc 100644 --- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/expressions/Expression.java +++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/style/expressions/Expression.java @@ -4,7 +4,6 @@ import android.support.annotation.ColorInt; import android.support.annotation.NonNull; import android.support.annotation.Nullable; import android.support.annotation.Size; - import com.google.gson.Gson; import com.google.gson.JsonArray; import com.google.gson.JsonElement; @@ -180,6 +179,8 @@ public class Expression { public static Expression literal(@NonNull Object object) { if (object.getClass().isArray()) { return literal(ExpressionArray.toObjectArray(object)); + } else if (object instanceof Expression) { + throw new RuntimeException("Can't convert an expression to a literal"); } return new ExpressionLiteral(object); } @@ -426,7 +427,7 @@ public class Expression { * @see <a href="https://www.mapbox.com/mapbox-gl-js/style-spec/#expressions-==">Style specification</a> */ public static Expression eq(@NonNull Expression compareOne, @NonNull String compareTwo) { - return eq(literal(compareOne), literal(compareTwo)); + return eq(compareOne, literal(compareTwo)); } /** @@ -449,7 +450,7 @@ public class Expression { * @see <a href="https://www.mapbox.com/mapbox-gl-js/style-spec/#expressions-==">Style specification</a> */ public static Expression eq(@NonNull Expression compareOne, @NonNull Number compareTwo) { - return eq(literal(compareOne), literal(compareTwo)); + return eq(compareOne, literal(compareTwo)); } /** @@ -496,7 +497,7 @@ public class Expression { * @see <a href="https://www.mapbox.com/mapbox-gl-js/style-spec/#expressions-!=">Style specification</a> */ public static Expression neq(Expression compareOne, boolean compareTwo) { - return new Expression("!=", literal(compareOne), literal(compareTwo)); + return new Expression("!=", compareOne, literal(compareTwo)); } /** @@ -519,7 +520,7 @@ public class Expression { * @see <a href="https://www.mapbox.com/mapbox-gl-js/style-spec/#expressions-!=">Style specification</a> */ public static Expression neq(@NonNull Expression compareOne, @NonNull String compareTwo) { - return new Expression("!=", literal(compareOne), literal(compareTwo)); + return new Expression("!=", compareOne, literal(compareTwo)); } /** @@ -542,7 +543,7 @@ public class Expression { * @see <a href="https://www.mapbox.com/mapbox-gl-js/style-spec/#expressions-!=">Style specification</a> */ public static Expression neq(@NonNull Expression compareOne, @NonNull Number compareTwo) { - return new Expression("!=", literal(compareOne), literal(compareTwo)); + return new Expression("!=", compareOne, literal(compareTwo)); } /** @@ -589,7 +590,7 @@ public class Expression { * @see <a href="https://www.mapbox.com/mapbox-gl-js/style-spec/#expressions-%3E">Style specification</a> */ public static Expression gt(@NonNull Expression compareOne, @NonNull Number compareTwo) { - return new Expression(">", literal(compareOne), literal(compareTwo)); + return new Expression(">", compareOne, literal(compareTwo)); } /** @@ -612,7 +613,7 @@ public class Expression { * @see <a href="https://www.mapbox.com/mapbox-gl-js/style-spec/#expressions-%3E">Style specification</a> */ public static Expression gt(@NonNull Expression compareOne, @NonNull String compareTwo) { - return new Expression(">", literal(compareOne), literal(compareTwo)); + return new Expression(">", compareOne, literal(compareTwo)); } /** @@ -659,7 +660,7 @@ public class Expression { * @see <a href="https://www.mapbox.com/mapbox-gl-js/style-spec/#expressions-%3C">Style specification</a> */ public static Expression lt(@NonNull Expression compareOne, @NonNull Number compareTwo) { - return new Expression("<", literal(compareOne), literal(compareTwo)); + return new Expression("<", compareOne, literal(compareTwo)); } /** @@ -682,7 +683,7 @@ public class Expression { * @see <a href="https://www.mapbox.com/mapbox-gl-js/style-spec/#expressions-%3C">Style specification</a> */ public static Expression lt(@NonNull Expression compareOne, @NonNull String compareTwo) { - return new Expression("<", literal(compareOne), literal(compareTwo)); + return new Expression("<", compareOne, literal(compareTwo)); } /** @@ -729,7 +730,7 @@ public class Expression { * @see <a href="https://www.mapbox.com/mapbox-gl-js/style-spec/#expressions-%3E%3D">Style specification</a> */ public static Expression gte(@NonNull Expression compareOne, @NonNull Number compareTwo) { - return new Expression(">=", literal(compareOne), literal(compareTwo)); + return new Expression(">=", compareOne, literal(compareTwo)); } /** @@ -752,7 +753,7 @@ public class Expression { * @see <a href="https://www.mapbox.com/mapbox-gl-js/style-spec/#expressions-%3E%3D">Style specification</a> */ public static Expression gte(@NonNull Expression compareOne, @NonNull String compareTwo) { - return new Expression(">=", literal(compareOne), literal(compareTwo)); + return new Expression(">=", compareOne, literal(compareTwo)); } /** @@ -799,7 +800,7 @@ public class Expression { * @see <a href="https://www.mapbox.com/mapbox-gl-js/style-spec/#expressions-%3C%3D">Style specification</a> */ public static Expression lte(@NonNull Expression compareOne, @NonNull Number compareTwo) { - return new Expression("<=", literal(compareOne), literal(compareTwo)); + return new Expression("<=", compareOne, literal(compareTwo)); } /** @@ -822,7 +823,7 @@ public class Expression { * @see <a href="https://www.mapbox.com/mapbox-gl-js/style-spec/#expressions-%3C%3D">Style specification</a> */ public static Expression lte(@NonNull Expression compareOne, @NonNull String compareTwo) { - return new Expression("<=", literal(compareOne), literal(compareTwo)); + return new Expression("<=", compareOne, literal(compareTwo)); } /** |