[core] Move RenderData alongside RenderItem

src/mbgl/renderer/painter.hpp

@@ -5,18 +5,17 @@
 #include <mbgl/tile/tile_id.hpp>
 
 #include <mbgl/renderer/frame_history.hpp>
+#include <mbgl/renderer/render_item.hpp>
 #include <mbgl/renderer/bucket.hpp>
 
 #include <mbgl/geometry/vao.hpp>
 #include <mbgl/geometry/static_vertex_buffer.hpp>
 
 #include <mbgl/gl/gl_config.hpp>
-
-#include <mbgl/style/render_item.hpp>
-#include <mbgl/style/style.hpp>
-
 #include <mbgl/gl/gl.hpp>
 
+#include <mbgl/style/style.hpp>
+
 #include <mbgl/util/noncopyable.hpp>
 #include <mbgl/util/chrono.hpp>
 #include <mbgl/util/constants.hpp>

src/mbgl/renderer/render_item.hpp

@@ -0,0 +1,38 @@
+#pragma once
+
+#include <mbgl/util/color.hpp>
+
+#include <set>
+#include <vector>
+
+namespace mbgl {
+
+class Tile;
+class Bucket;
+
+namespace style {
+class Layer;
+class Source;
+}
+
+class RenderItem {
+public:
+    RenderItem(const style::Layer& layer_,
+               const Tile* tile_ = nullptr,
+               Bucket* bucket_ = nullptr)
+        : tile(tile_), bucket(bucket_), layer(layer_) {
+    }
+
+    const Tile* const tile;
+    Bucket* const bucket;
+    const style::Layer& layer;
+};
+
+class RenderData {
+public:
+    Color backgroundColor = {{ 0, 0, 0, 0 }};
+    std::set<style::Source*> sources;
+    std::vector<RenderItem> order;
+};
+
+} // namespace mbgl

src/mbgl/style/render_item.hpp

@@ -1,24 +0,0 @@
-#pragma once
-
-namespace mbgl {
-
-class Tile;
-class Bucket;
-
-namespace style {
-class Layer;
-}
-
-struct RenderItem {
-    inline RenderItem(const style::Layer& layer_,
-                      const Tile* tile_ = nullptr,
-                      Bucket* bucket_ = nullptr)
-        : tile(tile_), bucket(bucket_), layer(layer_) {
-    }
-
-    const Tile* const tile;
-    Bucket* const bucket;
-    const style::Layer& layer;
-};
-
-} // namespace mbgl

src/mbgl/style/style.cpp

@@ -18,6 +18,7 @@
 #include <mbgl/style/update_parameters.hpp>
 #include <mbgl/style/cascade_parameters.hpp>
 #include <mbgl/style/calculation_parameters.hpp>
+#include <mbgl/renderer/render_item.hpp>
 #include <mbgl/geometry/glyph_atlas.hpp>
 #include <mbgl/geometry/line_atlas.hpp>
 #include <mbgl/util/constants.hpp>

src/mbgl/style/style.hpp

@@ -1,8 +1,6 @@
 #pragma once
 
-#include <mbgl/style/render_item.hpp>
 #include <mbgl/style/transition_options.hpp>
-
 #include <mbgl/style/observer.hpp>
 #include <mbgl/style/source_observer.hpp>
 #include <mbgl/text/glyph_store_observer.hpp>
@@ -15,12 +13,10 @@
 #include <mbgl/util/worker.hpp>
 #include <mbgl/util/optional.hpp>
 #include <mbgl/util/feature.hpp>
-#include <mbgl/util/color.hpp>
 
 #include <cstdint>
 #include <string>
 #include <vector>
-#include <set>
 
 namespace mbgl {
 
@@ -28,12 +24,7 @@ class FileSource;
 class GlyphAtlas;
 class SpriteAtlas;
 class LineAtlas;
-
-struct RenderData {
-    Color backgroundColor = {{ 0, 0, 0, 0 }};
-    std::set<style::Source*> sources;
-    std::vector<RenderItem> order;
-};
+class RenderData;
 
 namespace style {