use rustc_hir::HirIdSet;
use rustc_lint::{LateContext, LateLintPass, LintContext};
use rustc_middle::lint::in_external_macro;
-use rustc_middle::ty::TyS;
use rustc_session::{declare_tool_lint, impl_lint_pass};
use rustc_span::sym;
declare_clippy_lint! {
/// ### What it does
- /// Checks for types with a `fn new() -> Self` method and no
+ /// Checks for public types with a `pub fn new() -> Self` method and no
/// implementation of
/// [`Default`](https://doc.rust-lang.org/std/default/trait.Default.html).
///
///
/// ### Example
/// ```ignore
- /// struct Foo(Bar);
+ /// pub struct Foo(Bar);
///
/// impl Foo {
- /// fn new() -> Self {
+ /// pub fn new() -> Self {
/// Foo(Bar::new())
/// }
/// }
/// To fix the lint, add a `Default` implementation that delegates to `new`:
///
/// ```ignore
- /// struct Foo(Bar);
+ /// pub struct Foo(Bar);
///
/// impl Default for Foo {
/// fn default() -> Self {
/// }
/// }
/// ```
+ #[clippy::version = "pre 1.29.0"]
pub NEW_WITHOUT_DEFAULT,
style,
- "`fn new() -> Self` method without `Default` implementation"
+ "`pub fn new() -> Self` method without `Default` implementation"
}
#[derive(Clone, Default)]
}) = item.kind
{
for assoc_item in items {
- if let hir::AssocItemKind::Fn { has_self: false } = assoc_item.kind {
+ if assoc_item.kind == (hir::AssocItemKind::Fn { has_self: false }) {
let impl_item = cx.tcx.hir().impl_item(assoc_item.id);
if in_external_macro(cx.sess(), impl_item.span) {
return;
// can't be implemented for unsafe new
return;
}
+ if clippy_utils::is_doc_hidden(cx.tcx.hir().attrs(id)) {
+ // shouldn't be implemented when it is hidden in docs
+ return;
+ }
if impl_item
.generics
.params
if sig.decl.inputs.is_empty();
if name == sym::new;
if cx.access_levels.is_reachable(impl_item.def_id);
- let self_def_id = cx.tcx.hir().local_def_id(cx.tcx.hir().get_parent_item(id));
+ let self_def_id = cx.tcx.hir().get_parent_item(id);
let self_ty = cx.tcx.type_of(self_def_id);
- if TyS::same_type(self_ty, return_ty(cx, id));
+ if self_ty == return_ty(cx, id);
if let Some(default_trait_id) = cx.tcx.get_diagnostic_item(sym::Default);
then {
if self.impling_types.is_none() {