1 files changed, 82 insertions, 65 deletions

diff --git a/platform/android/src/style/android_conversion.hpp b/platform/android/src/style/android_conversion.hpp
index 082fe411e2..510a9f8444 100644
--- a/platform/android/src/style/android_conversion.hpp
+++ b/platform/android/src/style/android_conversion.hpp
@@ -4,8 +4,9 @@
 
 #include <mbgl/util/feature.hpp>
 #include <mbgl/util/logging.hpp>
-#include <mbgl/style/conversion.hpp>
 #include <mbgl/util/optional.hpp>
+#include <mbgl/style/conversion.hpp>
+#include <mbgl/style/conversion/geojson.hpp>
 
 #include <jni/jni.hpp>
 
@@ -13,89 +14,105 @@ namespace mbgl {
 namespace style {
 namespace conversion {
 
-inline bool isUndefined(const mbgl::android::Value& value) {
-    return value.isNull();
-}
+template <>
+class ConversionTraits<mbgl::android::Value> {
+public:
+    static bool isUndefined(const mbgl::android::Value& value) {
+        return value.isNull();
+    }
 
-inline bool isArray(const mbgl::android::Value& value) {
-    return value.isArray();
-}
+    static bool isArray(const mbgl::android::Value& value) {
+        return value.isArray();
+    }
 
-inline bool isObject(const mbgl::android::Value& value) {
-    return value.isObject();
-}
+    static bool isObject(const mbgl::android::Value& value) {
+        return value.isObject();
+    }
 
-inline std::size_t arrayLength(const mbgl::android::Value& value) {
-    return value.getLength();;
-}
+    static std::size_t arrayLength(const mbgl::android::Value& value) {
+        return value.getLength();;
+    }
 
-inline mbgl::android::Value arrayMember(const mbgl::android::Value& value, std::size_t i) {
-    return value.get(i);
-}
+    static mbgl::android::Value arrayMember(const mbgl::android::Value& value, std::size_t i) {
+        return value.get(i);
+    }
 
-inline optional<mbgl::android::Value> objectMember(const mbgl::android::Value& value, const char* key) {
-    mbgl::android::Value member = value.get(key);
+    static optional<mbgl::android::Value> objectMember(const mbgl::android::Value& value, const char* key) {
+        mbgl::android::Value member = value.get(key);
+        if (!member.isNull()) {
+            return member;
+        } else {
+            return {};
+        }
+    }
 
-    if (!member.isNull()) {
-        return member;
-    } else {
+    template <class Fn>
+    static optional<Error> eachMember(const mbgl::android::Value&, Fn&&) {
+        // TODO
+        mbgl::Log::Warning(mbgl::Event::Android, "eachMember not implemented");
         return {};
     }
-}
 
-template <class Fn>
-optional<Error> eachMember(const mbgl::android::Value&, Fn&&) {
-    // TODO
-    mbgl::Log::Warning(mbgl::Event::Android, "eachMember not implemented");
-    return {};
-}
+    static optional<bool> toBool(const mbgl::android::Value& value) {
+        if (value.isBool()) {
+            return value.toBool();
+        } else {
+            return {};
+        }
+    }
 
-inline optional<bool> toBool(const mbgl::android::Value& value) {
-    if (value.isBool()) {
-        return value.toBool();
-    } else {
-        return {};
+    static optional<float> toNumber(const mbgl::android::Value& value) {
+        if (value.isNumber()) {
+            auto num = value.toFloat();
+            return num;
+        } else {
+            return {};
+        }
     }
-}
 
-inline optional<float> toNumber(const mbgl::android::Value& value) {
-    if (value.isNumber()) {
-        auto num = value.toFloat();
-        return num;
-    } else {
-        return {};
+    static optional<double> toDouble(const mbgl::android::Value& value) {
+        if (value.isNumber()) {
+            return value.toDouble();
+        } else {
+            return {};
+        }
     }
-}
 
-inline optional<double> toDouble(const mbgl::android::Value& value) {
-    if (value.isNumber()) {
-        return value.toDouble();
-    } else {
-        return {};
+    static optional<std::string> toString(const mbgl::android::Value& value) {
+        if (value.isString()) {
+            return value.toString();
+        } else {
+            return {};
+        }
     }
-}
 
-inline optional<std::string> toString(const mbgl::android::Value& value) {
-    if (value.isString()) {
-        return value.toString();
-    } else {
-        return {};
+    static optional<Value> toValue(const mbgl::android::Value& value) {
+        if (value.isNull()) {
+            return {};
+        } else if (value.isBool()) {
+            return { value.toBool() };
+        } else if (value.isString()) {
+            return { value.toString() };
+        } else if (value.isNumber()) {
+            auto doubleVal = value.toDouble();
+            return { doubleVal - (int) doubleVal > 0.0 ? doubleVal : value.toLong() };
+        } else {
+            return {};
+        }
     }
-}
 
-inline optional<Value> toValue(const mbgl::android::Value& value) {
-    if (value.isNull()) {
-        return {};
-    } else if (value.isBool()) {
-        return { value.toBool() };
-    } else if (value.isString()) {
-        return { value.toString() };
-    } else if (value.isNumber()) {
-        auto doubleVal = value.toDouble();
-        return { doubleVal - (int) doubleVal > 0.0 ? doubleVal : value.toLong() };
-    } else {
-        return {};
+    static optional<GeoJSON> toGeoJSON(const mbgl::android::Value &value, Error &error) {
+        if (value.isNull() || !value.isString()) {
+            error = { "no json data found" };
+            return {};
+        }
+        return parseGeoJSON(value.toString(), error);
     }
+};
+
+template <class T, class...Args>
+optional<T> convert(mbgl::android::Value&& value, Error& error, Args&&...args) {
+    return convert<T>(Convertible(std::move(value)), error, std::forward<Args>(args)...);
 }
 
 } // namespace conversion