default_union_representation: Fix the order of short-circuit operations

rust-lang · Jan 26, 2022 · c07dfd8 · c07dfd8
1 parent 1028599
commit c07dfd8
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/clippy_lints/src/default_union_representation.rs b/clippy_lints/src/default_union_representation.rs
@@ -52,7 +52,7 @@ declare_lint_pass!(DefaultUnionRepresentation => [DEFAULT_UNION_REPRESENTATION])
 
 impl<'tcx> LateLintPass<'tcx> for DefaultUnionRepresentation {
     fn check_item(&mut self, cx: &LateContext<'tcx>, item: &'tcx Item<'tcx>) {
-        if !has_c_repr_attr(cx, item.hir_id()) && is_union_with_two_non_zst_fields(cx, item) {
+        if is_union_with_two_non_zst_fields(cx, item) && !has_c_repr_attr(cx, item.hir_id()) {
             span_lint_and_help(
                 cx,
                 DEFAULT_UNION_REPRESENTATION,