Skip to content

Commit a854ecc

Browse files
foskey51codetyri0n
authored andcommitted
chore: enforce clippy lint needless_pass_by_value to physical-expr-common (apache#18556)
## Which issue does this PR close? - Closes apache#18543 ## What changes are included in this PR? enforce clippy lint `needless_pass_by_value` to `datafusion-physical-expr-common` ## Are these changes tested? yes ## Are there any user-facing changes? no
1 parent cd849d7 commit a854ecc

File tree

1 file changed

+3
-0
lines changed
  • datafusion/physical-expr-common/src

1 file changed

+3
-0
lines changed

datafusion/physical-expr-common/src/lib.rs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,9 @@
2323
// Make sure fast / cheap clones on Arc are explicit:
2424
// https://github.com/apache/datafusion/issues/11143
2525
#![deny(clippy::clone_on_ref_ptr)]
26+
// https://github.com/apache/datafusion/issues/18503
27+
#![deny(clippy::needless_pass_by_value)]
28+
#![cfg_attr(test, allow(clippy::needless_pass_by_value))]
2629

2730
//! Physical Expr Common packages for [DataFusion]
2831
//! This package contains high level PhysicalExpr trait

0 commit comments

Comments
 (0)