This is an automated email from the ASF dual-hosted git repository.

allengeorge pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/thrift.git


The following commit(s) were added to refs/heads/master by this push:
     new 7e90252  THRIFT-4764: Use new syntax for clippy skipping Client: rs
     new 3b56a03  Merge pull request #2076 from allengeorge/thrift-4764
7e90252 is described below

commit 7e90252af64b0a0dee5e5e6f37808aa11a6b2e03
Author: Mikail Bagishov <[email protected]>
AuthorDate: Tue Jan 29 19:51:42 2019 +0300

    THRIFT-4764: Use new syntax for clippy skipping
    Client: rs
---
 compiler/cpp/src/thrift/generate/t_rs_generator.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/compiler/cpp/src/thrift/generate/t_rs_generator.cc 
b/compiler/cpp/src/thrift/generate/t_rs_generator.cc
index 70e1847..b240d99 100644
--- a/compiler/cpp/src/thrift/generate/t_rs_generator.cc
+++ b/compiler/cpp/src/thrift/generate/t_rs_generator.cc
@@ -547,7 +547,7 @@ void t_rs_generator::render_attributes_and_includes() {
   f_gen_ << "#![allow(unused_extern_crates)]" << endl;
   // constructors take *all* struct parameters, which can trigger the "too 
many arguments" warning
   // some auto-gen'd types can be deeply nested. clippy recommends factoring 
them out which is hard to autogen
-  f_gen_ << "#![cfg_attr(feature = \"cargo-clippy\", allow(too_many_arguments, 
type_complexity))]" << endl;
+  f_gen_ << "#![allow(clippy::too_many_arguments, clippy::type_complexity)]" 
<< endl;
   // prevent rustfmt from running against this file
   // lines are too long, code is (thankfully!) not visual-indented, etc.
   f_gen_ << "#![cfg_attr(rustfmt, rustfmt_skip)]" << endl;

Reply via email to