121e87bf14
`CheckAttrVisitor::check_doc_keyword` checks `#[doc(keyword = "..")]` attributes to ensure they are on an empty module, and that the value is a non-empty identifier. The `rustc::existing_doc_keyword` lint checks these attributes to ensure that the value is the name of a keyword. It's silly to have two different checking mechanisms for these attributes. This commit does the following. - Changes `check_doc_keyword` to check that the value is the name of a keyword (avoiding the need for the identifier check, which removes a dependency on `rustc_lexer`). - Removes the lint. - Updates tests accordingly. There is one hack: the `SelfTy` FIXME case used to used to be handled by disabling the lint, but now is handled with a special case in `is_doc_keyword`. That hack will go away if/when the FIXME is fixed. Co-Authored-By: Guillaume Gomez <guillaume1.gomez@gmail.com>
22 lines
837 B
Rust
22 lines
837 B
Rust
#![crate_name = "foo"]
|
|
|
|
#![feature(rustdoc_internals)]
|
|
|
|
//@ has foo/index.html '//h2[@id="keywords"]' 'Keywords'
|
|
//@ has foo/index.html '//a[@href="keyword.match.html"]' 'match'
|
|
//@ has foo/index.html '//div[@class="sidebar-elems"]//li/a' 'Keywords'
|
|
//@ has foo/index.html '//div[@class="sidebar-elems"]//li/a/@href' '#keywords'
|
|
//@ has foo/keyword.match.html '//h1' 'Keyword match'
|
|
//@ has foo/keyword.match.html '//section[@id="main-content"]//div[@class="docblock"]//p' 'this is a test!'
|
|
//@ has foo/index.html '//a/@href' '../foo/index.html'
|
|
//@ !has foo/foo/index.html
|
|
//@ !has-dir foo/foo
|
|
//@ !has foo/index.html '//span' '🔒'
|
|
#[doc(keyword = "match")]
|
|
/// this is a test!
|
|
mod foo{}
|
|
|
|
//@ has foo/keyword.break.html '//section[@id="main-content"]//div[@class="docblock"]//p' 'hello'
|
|
#[doc(keyword = "break")]
|
|
/// hello
|
|
mod bar {}
|