Skip to content

Conversation

@Kimundi
Copy link
Contributor

@Kimundi Kimundi commented Mar 30, 2014

They required unnecessary temporaries, are replaced with iterators, and would conflict with a possible future Iterable trait.

bors added a commit that referenced this pull request Mar 30, 2014
They required unnecessary temporaries, are replaced with iterators, and would conflict with a possible future `Iterable` trait.
@bors bors closed this Mar 30, 2014
@bors bors merged commit c356e3b into rust-lang:master Mar 30, 2014
Jarcho pushed a commit to Jarcho/rust that referenced this pull request Aug 24, 2024
…ct_constructor_on_adt, r=Alexendoo

Respect allow `inconsistent_struct_constructor` on the struct definition

Closes rust-lang#13203

Now we check if the target type is marked with `#[allow(clippy:inconsistent_struct_constructor)]` before lining.
As a side-effect of this change, The rule in the subject no longer runs on non-local `AdtDef`s. However, as suggested by `@Jarcho` it shouldn't be a big deal since most of the time we didn't have access to this information anyway.

> You can't get lint attributes from other crates. I would probably just restrict the lint to only work with types from the current crate while you're at it. Upstream crates don't have a definition order from the point of view of the current crate (with the exception of #[repr(C)] structs).

changelog: Respect allow `inconsistent_struct_constructor` on the struct definition.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants