mirror of
https://github.com/rust-lang/rust-clippy
synced 2024-11-23 21:23:56 +00:00
Auto merge of #5055 - krishna-veerareddy:fix-new-lint-generator-import, r=llogiq
Fix new lint generator import Fix `rustc_lint` import path and add lifetimes to `LateLintPass` in `new_lint` command. changelog: none
This commit is contained in:
commit
6bd0580887
1 changed files with 8 additions and 5 deletions
|
@ -11,9 +11,9 @@ pub fn create(pass: Option<&str>, lint_name: Option<&str>, category: Option<&str
|
|||
|
||||
match open_files(lint_name) {
|
||||
Ok((mut test_file, mut lint_file)) => {
|
||||
let (pass_type, pass_import, context_import) = match pass {
|
||||
"early" => ("EarlyLintPass", "use syntax::ast::*;", "EarlyContext"),
|
||||
"late" => ("LateLintPass", "use rustc_hir::*;", "LateContext"),
|
||||
let (pass_type, pass_lifetimes, pass_import, context_import) = match pass {
|
||||
"early" => ("EarlyLintPass", "", "use syntax::ast::*;", "EarlyContext"),
|
||||
"late" => ("LateLintPass", "<'_, '_>", "use rustc_hir::*;", "LateContext"),
|
||||
_ => {
|
||||
unreachable!("`pass_type` should only ever be `early` or `late`!");
|
||||
},
|
||||
|
@ -31,6 +31,7 @@ pub fn create(pass: Option<&str>, lint_name: Option<&str>, category: Option<&str
|
|||
if let Err(e) = lint_file.write_all(
|
||||
get_lint_file_contents(
|
||||
pass_type,
|
||||
pass_lifetimes,
|
||||
lint_name,
|
||||
&camel_case_name,
|
||||
category,
|
||||
|
@ -125,6 +126,7 @@ fn main() {{
|
|||
|
||||
fn get_lint_file_contents(
|
||||
pass_type: &str,
|
||||
pass_lifetimes: &str,
|
||||
lint_name: &str,
|
||||
camel_case_name: &str,
|
||||
category: &str,
|
||||
|
@ -132,7 +134,7 @@ fn get_lint_file_contents(
|
|||
context_import: &str,
|
||||
) -> String {
|
||||
format!(
|
||||
"use rustc::lint::{{LintArray, LintPass, {type}, {context_import}}};
|
||||
"use rustc_lint::{{LintArray, LintPass, {type}, {context_import}}};
|
||||
use rustc_session::{{declare_lint_pass, declare_tool_lint}};
|
||||
{pass_import}
|
||||
|
||||
|
@ -155,9 +157,10 @@ declare_clippy_lint! {{
|
|||
|
||||
declare_lint_pass!({name_camel} => [{name_upper}]);
|
||||
|
||||
impl {type} for {name_camel} {{}}
|
||||
impl {type}{lifetimes} for {name_camel} {{}}
|
||||
",
|
||||
type=pass_type,
|
||||
lifetimes=pass_lifetimes,
|
||||
name_upper=lint_name.to_uppercase(),
|
||||
name_camel=camel_case_name,
|
||||
category=category,
|
||||
|
|
Loading…
Reference in a new issue