[core] Rationalize naming for style-related code

include/mbgl/annotation/annotation.hpp

@@ -1,9 +1,8 @@
 #pragma once
 
-#include <mbgl/style/types.hpp>
-
 #include <mbgl/util/geometry.hpp>
 #include <mbgl/util/variant.hpp>
+#include <mbgl/util/color.hpp>
 
 #include <cstdint>
 #include <vector>

include/mbgl/map/map.hpp

@@ -9,7 +9,7 @@
 #include <mbgl/util/feature.hpp>
 #include <mbgl/util/noncopyable.hpp>
 #include <mbgl/annotation/annotation.hpp>
-#include <mbgl/style/property_transition.hpp>
+#include <mbgl/style/transition_options.hpp>
 
 #include <cstdint>
 #include <string>
@@ -22,10 +22,13 @@ namespace mbgl {
 class FileSource;
 class View;
 class SpriteImage;
-class Layer;
 struct CameraOptions;
 struct AnimationOptions;
 
+namespace style {
+class Layer;
+}
+
 class Map : private util::noncopyable {
 public:
     explicit Map(View&, FileSource&,
@@ -47,9 +50,9 @@ class Map : private util::noncopyable {
     void update(Update update);
 
     // Styling
-    void addClass(const std::string&, const PropertyTransition& = {});
-    void removeClass(const std::string&, const PropertyTransition& = {});
-    void setClasses(const std::vector<std::string>&, const PropertyTransition& = {});
+    void addClass(const std::string&, const style::TransitionOptions& = {});
+    void removeClass(const std::string&, const style::TransitionOptions& = {});
+    void setClasses(const std::vector<std::string>&, const style::TransitionOptions& = {});
 
     bool hasClass(const std::string&) const;
     std::vector<std::string> getClasses() const;
@@ -146,7 +149,7 @@ class Map : private util::noncopyable {
 
     AnnotationIDs getPointAnnotationsInBounds(const LatLngBounds&);
 
-    void addLayer(std::unique_ptr<Layer>, const optional<std::string>& beforeLayerID = {});
+    void addLayer(std::unique_ptr<style::Layer>, const optional<std::string>& beforeLayerID = {});
     void removeLayer(const std::string& layerID);
 
     // Feature queries

include/mbgl/storage/offline.hpp

@@ -12,7 +12,7 @@
 namespace mbgl {
 
 class TileID;
-class SourceInfo;
+class Tileset;
 
 /*
  * An offline region defined by a style URL, geographic bounding box, zoom range, and
@@ -30,7 +30,7 @@ class OfflineTilePyramidRegionDefinition {
     OfflineTilePyramidRegionDefinition(const std::string&, const LatLngBounds&, double, double, float);
 
     /* Private */
-    std::vector<CanonicalTileID> tileCover(SourceType, uint16_t tileSize, const SourceInfo&) const;
+    std::vector<CanonicalTileID> tileCover(SourceType, uint16_t tileSize, const Tileset&) const;
 
     const std::string styleURL;
     const LatLngBounds bounds;

include/mbgl/storage/resource.hpp

@@ -2,7 +2,7 @@
 
 #include <mbgl/storage/response.hpp>
 #include <mbgl/util/optional.hpp>
-#include <mbgl/style/types.hpp>
+#include <mbgl/util/font_stack.hpp>
 
 #include <string>
 

include/mbgl/style/filter.hpp

@@ -7,6 +7,7 @@
 #include <vector>
 
 namespace mbgl {
+namespace style {
 
 typedef variant<
     class NullFilter,
@@ -100,4 +101,5 @@ class NotHasFilter {
     std::string key;
 };
 
+} // namespace style
 } // namespace mbgl

include/mbgl/style/function.hpp

@@ -0,0 +1,27 @@
+#pragma once
+
+#include <vector>
+#include <utility>
+
+namespace mbgl {
+namespace style {
+
+template <typename T>
+class Function {
+public:
+    using Stop = std::pair<float, T>;
+    using Stops = std::vector<Stop>;
+
+    explicit Function(const Stops& stops_, float base_)
+        : base(base_), stops(stops_) {}
+
+    float getBase() const { return base; }
+    const std::vector<std::pair<float, T>>& getStops() const { return stops; }
+
+private:
+    float base = 1;
+    std::vector<std::pair<float, T>> stops;
+};
+
+} // namespace style
+} // namespace mbgl

include/mbgl/style/layer.hpp

@@ -1,12 +1,12 @@
-#ifndef MBGL_LAYER
-#define MBGL_LAYER
+#pragma once
 
 #include <mbgl/util/noncopyable.hpp>
 #include <mbgl/style/types.hpp>
 
 #include <memory>
 
 namespace mbgl {
+namespace style {
 
 /**
  * The runtime representation of a [layer](https://www.mapbox.com/mapbox-gl-style-spec/#layers) from the Mapbox Style
@@ -79,6 +79,5 @@ class Layer : public mbgl::util::noncopyable {
     Layer(Type, std::unique_ptr<Impl>);
 };
 
+} // namespace style
 } // namespace mbgl
-
-#endif

include/mbgl/layer/background_layer.hpp → include/mbgl/style/layers/background_layer.hpp

@@ -6,7 +6,10 @@
 #include <mbgl/style/filter.hpp>
 #include <mbgl/style/property_value.hpp>
 
+#include <mbgl/util/color.hpp>
+
 namespace mbgl {
+namespace style {
 
 class BackgroundLayer : public Layer {
 public:
@@ -38,4 +41,5 @@ inline bool Layer::is<BackgroundLayer>() const {
     return type == Type::Background;
 }
 
+} // namespace style
 } // namespace mbgl

include/mbgl/layer/circle_layer.hpp → include/mbgl/style/layers/circle_layer.hpp

@@ -6,7 +6,10 @@
 #include <mbgl/style/filter.hpp>
 #include <mbgl/style/property_value.hpp>
 
+#include <mbgl/util/color.hpp>
+
 namespace mbgl {
+namespace style {
 
 class CircleLayer : public Layer {
 public:
@@ -56,4 +59,5 @@ inline bool Layer::is<CircleLayer>() const {
     return type == Type::Circle;
 }
 
+} // namespace style
 } // namespace mbgl

include/mbgl/layer/custom_layer.hpp → include/mbgl/style/layers/custom_layer.hpp

@@ -3,6 +3,7 @@
 #include <mbgl/style/layer.hpp>
 
 namespace mbgl {
+namespace style {
 
 /**
  * Initialize any GL state needed by the custom layer. This method is called once, from the
@@ -66,4 +67,5 @@ inline bool Layer::is<CustomLayer>() const {
     return type == Type::Custom;
 }
 
+} // namespace style
 } // namespace mbgl

include/mbgl/layer/fill_layer.hpp → include/mbgl/style/layers/fill_layer.hpp

@@ -6,7 +6,10 @@
 #include <mbgl/style/filter.hpp>
 #include <mbgl/style/property_value.hpp>
 
+#include <mbgl/util/color.hpp>
+
 namespace mbgl {
+namespace style {
 
 class FillLayer : public Layer {
 public:
@@ -59,4 +62,5 @@ inline bool Layer::is<FillLayer>() const {
     return type == Type::Fill;
 }
 
+} // namespace style
 } // namespace mbgl

include/mbgl/layer/line_layer.hpp → include/mbgl/style/layers/line_layer.hpp

@@ -6,9 +6,12 @@
 #include <mbgl/style/filter.hpp>
 #include <mbgl/style/property_value.hpp>
 
+#include <mbgl/util/color.hpp>
+
 #include <vector>
 
 namespace mbgl {
+namespace style {
 
 class LineLayer : public Layer {
 public:
@@ -84,4 +87,5 @@ inline bool Layer::is<LineLayer>() const {
     return type == Type::Line;
 }
 
+} // namespace style
 } // namespace mbgl

include/mbgl/layer/raster_layer.hpp → include/mbgl/style/layers/raster_layer.hpp

@@ -6,7 +6,10 @@
 #include <mbgl/style/filter.hpp>
 #include <mbgl/style/property_value.hpp>
 
+#include <mbgl/util/color.hpp>
+
 namespace mbgl {
+namespace style {
 
 class RasterLayer : public Layer {
 public:
@@ -55,4 +58,5 @@ inline bool Layer::is<RasterLayer>() const {
     return type == Type::Raster;
 }
 
+} // namespace style
 } // namespace mbgl

include/mbgl/layer/symbol_layer.hpp → include/mbgl/style/layers/symbol_layer.hpp

@@ -6,9 +6,12 @@
 #include <mbgl/style/filter.hpp>
 #include <mbgl/style/property_value.hpp>
 
+#include <mbgl/util/color.hpp>
+
 #include <vector>
 
 namespace mbgl {
+namespace style {
 
 class SymbolLayer : public Layer {
 public:
@@ -177,4 +180,5 @@ inline bool Layer::is<SymbolLayer>() const {
     return type == Type::Symbol;
 }
 
+} // namespace style
 } // namespace mbgl

src/mbgl/style/property_value.hpp → include/mbgl/style/property_value.hpp

@@ -1,9 +1,10 @@
 #pragma once
 
 #include <mbgl/util/variant.hpp>
-#include <mbgl/style/types.hpp>
+#include <mbgl/style/function.hpp>
 
 namespace mbgl {
+namespace style {
 
 class Undefined {};
 
@@ -33,4 +34,5 @@ class PropertyValue {
     }
 };
 
-}
+} // namespace style
+} // namespace mbgl

include/mbgl/style/property_transition.hpp → include/mbgl/style/transition_options.hpp

@@ -4,14 +4,16 @@
 #include <mbgl/util/optional.hpp>
 
 namespace mbgl {
+namespace style {
 
-class PropertyTransition {
+class TransitionOptions {
 public:
-    PropertyTransition(const optional<Duration>& duration_ = {}, const optional<Duration>& delay_ = {})
+    TransitionOptions(const optional<Duration>& duration_ = {}, const optional<Duration>& delay_ = {})
         : duration(duration_), delay(delay_) {}
 
     optional<Duration> duration;
     optional<Duration> delay;
 };
 
+} // namespace style
 } // namespace mbgl

include/mbgl/style/types.hpp

@@ -2,44 +2,9 @@
 
 #include <mbgl/util/enum.hpp>
 
-#include <string>
-#include <array>
-#include <vector>
-#include <utility>
-
 namespace mbgl {
 
-// Stores a premultiplied color, with all four channels ranging from 0..1
-using Color = std::array<float, 4>;
-
-// An array of font names
-using FontStack = std::vector<std::string>;
-
-std::string fontStackToString(const FontStack&);
-
-struct FontStackHash {
-    std::size_t operator()(const FontStack&) const;
-};
-
-template <typename T>
-class Function {
-public:
-    using Stop = std::pair<float, T>;
-    using Stops = std::vector<Stop>;
-
-    explicit Function(const Stops& stops_, float base_)
-        : base(base_), stops(stops_) {}
-
-    float getBase() const { return base; }
-    const std::vector<std::pair<float, T>>& getStops() const { return stops; }
-
-private:
-    float base = 1;
-    std::vector<std::pair<float, T>> stops;
-};
-
-// -------------------------------------------------------------------------------------------------
-
+// TODO: should be in public source.hpp header and style namespace
 enum class SourceType : uint8_t {
     Vector,
     Raster,
@@ -56,7 +21,7 @@ MBGL_DEFINE_ENUM_CLASS(SourceTypeClass, SourceType, {
     { SourceType::Annotations, "annotations" },
 });
 
-// -------------------------------------------------------------------------------------------------
+namespace style {
 
 enum class VisibilityType : bool {
     Visible,
@@ -122,5 +87,5 @@ enum class TextTransformType : uint8_t {
     Lowercase,
 };
 
+} // namespace style
 } // namespace mbgl
-

include/mbgl/util/color.hpp

@@ -0,0 +1,10 @@
+#pragma once
+
+#include <array>
+
+namespace mbgl {
+
+// Stores a premultiplied color, with all four channels ranging from 0..1
+using Color = std::array<float, 4>;
+
+} // namespace mbgl

include/mbgl/util/font_stack.hpp

@@ -0,0 +1,17 @@
+#pragma once
+
+#include <string>
+#include <vector>
+
+namespace mbgl {
+
+// An array of font names
+using FontStack = std::vector<std::string>;
+
+std::string fontStackToString(const FontStack&);
+
+struct FontStackHash {
+    std::size_t operator()(const FontStack&) const;
+};
+
+} // namespace mbgl