[tidy] Check llvm-namespace-comment

Ref: http://clang.llvm.org/extra/clang-tidy/checks/llvm-namespace-comment.html

include/mbgl/map/map.hpp

@@ -27,7 +27,7 @@ struct AnimationOptions;
 
 namespace style {
 class Layer;
-}
+} // namespace style
 
 class Map : private util::noncopyable {
 public:

platform/default/mbgl/storage/offline_database.hpp

@@ -15,8 +15,8 @@ namespace mapbox {
 namespace sqlite {
 class Database;
 class Statement;
-}
-}
+} // namespace sqlite
+} // namespace mapbox
 
 namespace mbgl {
 

platform/default/mbgl/storage/offline_download.hpp

@@ -17,7 +17,7 @@ class Tileset;
 
 namespace style {
 class Parser;
-}
+} // namespace style
 
 /**
  * Coordinates the request and storage of all resources for an offline region.

scripts/build-version.py

@@ -67,8 +67,8 @@ extern const char *revision;
 extern const char *string;
 extern const unsigned int number;
 
-}} // namespace mbgl
 }} // namespace version
+}} // namespace mbgl
 """.format(
     major = tag[0],
     minor = tag[1],

src/mbgl/annotation/annotation_manager.hpp

@@ -21,7 +21,7 @@ class ShapeAnnotationImpl;
 
 namespace style {
 class Style;
-}
+} // namespace style
 
 class AnnotationManager : private util::noncopyable {
 public:

src/mbgl/annotation/annotation_tile.hpp

@@ -9,7 +9,7 @@ class AnnotationManager;
 
 namespace style {
 class UpdateParameters;
-}
+} // namespace style
 
 class AnnotationTile : public GeometryTile {
 public:

src/mbgl/annotation/shape_annotation_impl.hpp

@@ -14,7 +14,7 @@ class CanonicalTileID;
 
 namespace style {
 class Style;
-}
+} // namespace style
 
 class ShapeAnnotationImpl {
 public:

src/mbgl/geometry/feature_index.hpp

@@ -13,7 +13,7 @@ namespace mbgl {
 
 namespace style {
 class Style;
-}
+} // namespace style
 
 class CollisionTile;
 class CanonicalTileID;
@@ -73,4 +73,4 @@ private:
 
     std::unordered_map<std::string, std::vector<std::string>> bucketLayerIDs;
 };
-}
+} // namespace mbgl

src/mbgl/renderer/bucket.hpp

@@ -17,11 +17,11 @@ class CollisionTile;
 
 namespace gl {
 class ObjectStore;
-}
+} // namespace gl
 
 namespace style {
 class Layer;
-}
+} // namespace style
 
 class Bucket : private util::noncopyable {
 public:

src/mbgl/renderer/circle_bucket.cpp

@@ -100,4 +100,4 @@ void CircleBucket::drawCircles(CircleShader& shader, gl::ObjectStore& store) {
     }
 }
 
-}
+} // namespace mbgl

src/mbgl/renderer/debug_bucket.hpp

@@ -12,7 +12,7 @@ class PlainShader;
 
 namespace gl {
 class ObjectStore;
-}
+} // namespace gl
 
 class DebugBucket : private util::noncopyable {
 public:

src/mbgl/renderer/fill_bucket.cpp

@@ -21,8 +21,8 @@ template <> struct nth<0, mbgl::GeometryCoordinate> {
 template <> struct nth<1, mbgl::GeometryCoordinate> {
     inline static int64_t get(const mbgl::GeometryCoordinate& t) { return t.y; };
 };
-}
-}
+} // namespace util
+} // namespace mapbox
 
 namespace mbgl {
 
@@ -168,4 +168,4 @@ void FillBucket::drawVertices(OutlinePatternShader& shader, gl::ObjectStore& sto
     }
 }
 
-}
+} // namespace mbgl

src/mbgl/renderer/line_bucket.cpp

@@ -508,4 +508,4 @@ void LineBucket::drawLinePatterns(LinepatternShader& shader, gl::ObjectStore& st
     }
 }
 
-}
+} // namespace mbgl

src/mbgl/renderer/painter.cpp

@@ -310,4 +310,4 @@ void Painter::setDepthSublayer(int n) {
     config.depthRange = { nearDepth, farDepth };
 }
 
-}
+} // namespace mbgl

src/mbgl/renderer/painter.hpp

@@ -61,7 +61,7 @@ struct ClipID;
 
 namespace util {
 class ObjectStore;
-}
+} // namespace util
 
 namespace style {
 class Style;
@@ -72,7 +72,7 @@ class CircleLayer;
 class SymbolLayer;
 class RasterLayer;
 class BackgroundLayer;
-}
+} // namespace style
 
 struct FrameData {
     std::array<uint16_t, 2> framebufferSize;

src/mbgl/renderer/painter_background.cpp

@@ -110,4 +110,4 @@ void Painter::renderBackground(const BackgroundLayer& layer) {
     }
 }
 
-}
+} // namespace mbgl

src/mbgl/renderer/painter_circle.cpp

@@ -46,4 +46,4 @@ void Painter::renderCircle(CircleBucket& bucket,
     bucket.drawCircles(*circleShader, store);
 }
 
-}
+} // namespace mbgl

src/mbgl/renderer/painter_fill.cpp

@@ -198,4 +198,4 @@ void Painter::renderFill(FillBucket& bucket,
     }
 }
 
-}
+} // namespace mbgl

src/mbgl/renderer/painter_line.cpp

@@ -164,4 +164,4 @@ void Painter::renderLine(LineBucket& bucket,
     }
 }
 
-}
+} // namespace mbgl

src/mbgl/renderer/painter_raster.cpp

@@ -69,4 +69,4 @@ std::array<float, 3> Painter::spinWeights(float spin) {
     return spin_weights;
 }
 
-}
+} // namespace mbgl

src/mbgl/renderer/painter_symbol.cpp

@@ -271,4 +271,4 @@ void Painter::renderSymbol(SymbolBucket& bucket,
     config.activeTexture = GL_TEXTURE0;
 }
 
-}
+} // namespace mbgl

src/mbgl/renderer/raster_bucket.cpp

@@ -43,4 +43,4 @@ bool RasterBucket::needsClipping() const {
     return false;
 }
 
-}
+} // namespace mbgl

src/mbgl/renderer/render_item.hpp

@@ -13,7 +13,7 @@ class Bucket;
 namespace style {
 class Layer;
 class Source;
-}
+} // namespace style
 
 class RenderItem {
 public:

src/mbgl/storage/resource.cpp

@@ -86,4 +86,4 @@ Resource Resource::tile(const std::string& urlTemplate,
     };
 }
 
-}
+} // namespace mbgl

src/mbgl/style/filter_evaluator.hpp

@@ -159,5 +159,5 @@ private:
     const GeometryTileFeature& feature;
 };
 
-} // namespace mbgl
+} // namespace style
 } // namespace mbgl

src/mbgl/style/property_evaluator.hpp

@@ -61,6 +61,6 @@ namespace util {
 template <typename T>
 struct Interpolator<style::Faded<T>>
     : Uninterpolated {};
-}
+} // namespace util
 
 } // namespace mbgl

src/mbgl/text/quads.hpp

@@ -12,7 +12,7 @@ class PositionedIcon;
 
 namespace style {
 class SymbolLayoutProperties;
-}
+} // namespace style
 
 struct SymbolQuad {
     explicit SymbolQuad(const Point<float> &tl_, const Point<float> &tr_,

src/mbgl/text/shaping.hpp

@@ -11,7 +11,7 @@ struct SpriteAtlasElement;
 
 namespace style {
 class SymbolLayoutProperties;
-}
+} // namespace style
 
 class PositionedIcon {
     public:

src/mbgl/tile/geojson_tile.hpp

@@ -12,7 +12,7 @@ namespace mbgl {
 
 namespace style {
 class UpdateParameters;
-}
+} // namespace style
 
 class GeoJSONTile : public GeometryTile {
 public:

src/mbgl/tile/geometry_tile.hpp

@@ -17,7 +17,7 @@ class FeatureIndex;
 
 namespace style {
 class Style;
-}
+} // namespace style
 
 class GeometryTile : public Tile {
 public:

src/mbgl/tile/raster_tile.hpp

@@ -14,7 +14,7 @@ namespace gl { class TexturePool; }
 namespace style {
 class Layer;
 class UpdateParameters;
-}
+} // namespace style
 
 class RasterTile : public Tile {
 public:

src/mbgl/tile/tile.hpp

@@ -23,7 +23,7 @@ class TileObserver;
 
 namespace style {
 class Layer;
-}
+} // namespace style
 
 class Tile : private util::noncopyable {
 public:

src/mbgl/tile/tile_source.hpp

@@ -12,7 +12,7 @@ class Tileset;
 
 namespace style {
 class UpdateParameters;
-}
+} // namespace style
 
 template <typename T>
 class TileSource : private util::noncopyable {

src/mbgl/tile/tile_worker.cpp

@@ -188,4 +188,4 @@ void TileWorker::insertBucket(const std::string& name, std::unique_ptr<Bucket> b
     }
 }
 
-}
+} // namespace mbgl

src/mbgl/tile/tile_worker.hpp

@@ -27,7 +27,7 @@ class Bucket;
 namespace style {
 class Layer;
 class SymbolLayer;
-}
+} // namespace style
 
 // We're using this class to shuttle the resulting buckets from the worker thread to the MapContext
 // thread. This class is movable-only because the vector contains movable-only value elements.

src/mbgl/tile/vector_tile.hpp

@@ -9,7 +9,7 @@ class Tileset;
 
 namespace style {
 class UpdateParameters;
-}
+} // namespace style
 
 class VectorTile : public GeometryTile {
 public:

src/mbgl/util/grid_index.cpp

@@ -79,4 +79,4 @@ int32_t GridIndex<T>::convertToCellCoord(int32_t x) const {
 }
 
 template class GridIndex<IndexedSubfeature>;
-}
+} // namespace mbgl

src/mbgl/util/grid_index.hpp

@@ -35,4 +35,4 @@ private:
 
 };
 
-}
+} // namespace mbgl

src/mbgl/util/intersection_tests.cpp

@@ -143,5 +143,5 @@ bool multiPolygonIntersectsMultiPolygon(const GeometryCollection& multiPolygonA,
     return false;
 }
 
-}
-}
+} // namespace util
+} // namespace mbgl

src/mbgl/util/intersection_tests.hpp

@@ -9,5 +9,5 @@ bool multiPolygonIntersectsBufferedMultiPoint(const GeometryCollection&, const G
 bool multiPolygonIntersectsBufferedMultiLine(const GeometryCollection&, const GeometryCollection&, float radius);
 bool multiPolygonIntersectsMultiPolygon(const GeometryCollection&, const GeometryCollection&);
 
-}
+} // namespace util
 } // namespace mbgl

src/mbgl/util/utf.hpp

@@ -17,5 +17,5 @@ class utf8_to_utf32 {
     }
 };
 
-} // namespace mbgl
 } // namespace util
+} // namespace mbgl

test/include/mbgl/test.hpp

@@ -4,4 +4,4 @@ namespace mbgl {
 
 int runTests(int argc, char* argv[]);
 
-}
+} // namespace mbgl

test/src/mbgl/test/test.cpp

@@ -14,4 +14,4 @@ int runTests(int argc, char *argv[]) {
     return RUN_ALL_TESTS();
 }
 
-}
+} // namespace mbgl

test/src/mbgl/test/util.hpp

@@ -73,5 +73,5 @@ void checkImage(const std::string& base,
                 double imageThreshold = 0,
                 double pixelThreshold = 0);
 
-}
-}
+} // namespace test
+} // namespace mbgl

test/storage/offline_database.cpp

@@ -78,7 +78,7 @@ private:
     bool locked = false;
 };
 
-}
+} // namespace
 
 TEST(OfflineDatabase, TEST_REQUIRES_WRITE(Create)) {
     using namespace mbgl;