Fix compile error due to unused lambda capture (envoyproxy#50)

This commit fixes the following compilation error: ``` ERROR: /Users/marpaia/git/envoy-filter-example/http-filter-example/BUILD:37:1: C++ compilation of rule '//http-filter-example:http_filter_config' failed (Exit 1) http-filter-example/http_filter_config.cc:49:14: error: lambda capture 'context' is not used [-Werror,-Wunused-lambda-capture] return [&context, config](Http::FilterChainFactoryCallbacks& callbacks) -> void { ^ 1 error generated. Target //http-filter-example:envoy failed to build ``` This commit also adds Envoy's `.clang-format` file so that editors don't incorrectly format C++. Signed-off-by: Mike Arpaia <[email protected]>
billowqiu · May 7, 2018 · cfcdc8a · cfcdc8a
1 parent 72a1927
commit cfcdc8a
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/.clang-format b/.clang-format
@@ -0,0 +1,9 @@
+---
+Language:        Cpp
+AccessModifierOffset: -2
+ColumnLimit: 100
+DerivePointerAlignment: false
+PointerAlignment: Left
+SortIncludes: false
+...
+
diff --git a/http-filter-example/http_filter_config.cc b/http-filter-example/http_filter_config.cc
@@ -41,12 +41,12 @@ class HttpSampleDecoderFilterConfig : public NamedHttpFilterConfigFactory {
   std::string name() override { return "sample"; }
 
 private:
-  Http::FilterFactoryCb createFilter(const sample::Decoder& proto_config, FactoryContext& context) {
+  Http::FilterFactoryCb createFilter(const sample::Decoder& proto_config, FactoryContext&) {
     Http::HttpSampleDecoderFilterConfigSharedPtr config =
         std::make_shared<Http::HttpSampleDecoderFilterConfig>(
             Http::HttpSampleDecoderFilterConfig(proto_config));
 
-    return [&context, config](Http::FilterChainFactoryCallbacks& callbacks) -> void {
+    return [config](Http::FilterChainFactoryCallbacks& callbacks) -> void {
       auto filter = new Http::HttpSampleDecoderFilter(config);
       callbacks.addStreamDecoderFilter(Http::StreamDecoderFilterSharedPtr{filter});
     };