2021-01-01 11:28:10 +00:00
<!DOCTYPE HTML>
< html lang = "en" class = "sidebar-visible no-js rust" >
< head >
<!-- Book generated using mdBook -->
< meta charset = "UTF-8" >
< title > #[deny(warnings)] - Rust Design Patterns< / title >
<!-- Custom HTML head -->
< meta content = "text/html; charset=utf-8" http-equiv = "Content-Type" >
< meta name = "description" content = "A catalogue of Rust design patterns, anti-patterns and idioms" >
< meta name = "viewport" content = "width=device-width, initial-scale=1" >
< meta name = "theme-color" content = "#ffffff" / >
< link rel = "icon" href = "../favicon.svg" >
< link rel = "shortcut icon" href = "../favicon.png" >
< link rel = "stylesheet" href = "../css/variables.css" >
< link rel = "stylesheet" href = "../css/general.css" >
< link rel = "stylesheet" href = "../css/chrome.css" >
< link rel = "stylesheet" href = "../css/print.css" media = "print" >
<!-- Fonts -->
< link rel = "stylesheet" href = "../FontAwesome/css/font-awesome.css" >
< link rel = "stylesheet" href = "../fonts/fonts.css" >
<!-- Highlight.js Stylesheets -->
< link rel = "stylesheet" href = "../highlight.css" >
< link rel = "stylesheet" href = "../tomorrow-night.css" >
< link rel = "stylesheet" href = "../ayu-highlight.css" >
<!-- Custom theme stylesheets -->
< / head >
< body >
<!-- Provide site root to javascript -->
< script type = "text/javascript" >
var path_to_root = "../";
var default_theme = window.matchMedia("(prefers-color-scheme: dark)").matches ? "navy" : "rust";
< / script >
<!-- Work around some values being stored in localStorage wrapped in quotes -->
< script type = "text/javascript" >
try {
var theme = localStorage.getItem('mdbook-theme');
var sidebar = localStorage.getItem('mdbook-sidebar');
if (theme.startsWith('"') & & theme.endsWith('"')) {
localStorage.setItem('mdbook-theme', theme.slice(1, theme.length - 1));
}
if (sidebar.startsWith('"') & & sidebar.endsWith('"')) {
localStorage.setItem('mdbook-sidebar', sidebar.slice(1, sidebar.length - 1));
}
} catch (e) { }
< / script >
<!-- Set the theme before any content is loaded, prevents flash -->
< script type = "text/javascript" >
var theme;
try { theme = localStorage.getItem('mdbook-theme'); } catch(e) { }
if (theme === null || theme === undefined) { theme = default_theme; }
var html = document.querySelector('html');
html.classList.remove('no-js')
html.classList.remove('rust')
html.classList.add(theme);
html.classList.add('js');
< / script >
<!-- Hide / unhide sidebar before it is displayed -->
< script type = "text/javascript" >
var html = document.querySelector('html');
var sidebar = 'hidden';
if (document.body.clientWidth >= 1080) {
try { sidebar = localStorage.getItem('mdbook-sidebar'); } catch(e) { }
sidebar = sidebar || 'visible';
}
html.classList.remove('sidebar-visible');
html.classList.add("sidebar-" + sidebar);
< / script >
< nav id = "sidebar" class = "sidebar" aria-label = "Table of contents" >
< div class = "sidebar-scrollbox" >
2021-01-02 11:15:56 +00:00
< ol class = "chapter" > < li class = "chapter-item expanded " > < a href = "../intro.html" > < strong aria-hidden = "true" > 1.< / strong > Introduction< / a > < / li > < li class = "chapter-item expanded " > < a href = "../idioms/index.html" > < strong aria-hidden = "true" > 2.< / strong > Idioms< / a > < / li > < li > < ol class = "section" > < li class = "chapter-item expanded " > < a href = "../idioms/concat-format.html" > < strong aria-hidden = "true" > 2.1.< / strong > Concatenating Strings with format!< / a > < / li > < li class = "chapter-item expanded " > < a href = "../idioms/ctor.html" > < strong aria-hidden = "true" > 2.2.< / strong > Constructor< / a > < / li > < li class = "chapter-item expanded " > < a href = "../idioms/default.html" > < strong aria-hidden = "true" > 2.3.< / strong > The Default Trait< / a > < / li > < li class = "chapter-item expanded " > < a href = "../idioms/deref.html" > < strong aria-hidden = "true" > 2.4.< / strong > Collections Are Smart Pointers< / a > < / li > < li class = "chapter-item expanded " > < a href = "../idioms/dtor-finally.html" > < strong aria-hidden = "true" > 2.5.< / strong > Finalisation in Destructors< / a > < / li > < li class = "chapter-item expanded " > < a href = "../idioms/mem-replace.html" > < strong aria-hidden = "true" > 2.6.< / strong > mem::replace(_)< / a > < / li > < li class = "chapter-item expanded " > < a href = "../idioms/on-stack-dyn-dispatch.html" > < strong aria-hidden = "true" > 2.7.< / strong > On-Stack Dynamic Dispatch< / a > < / li > < li class = "chapter-item expanded " > < a href = "../idioms/option-iter.html" > < strong aria-hidden = "true" > 2.8.< / strong > Iterating over an Option< / a > < / li > < li class = "chapter-item expanded " > < a href = "../idioms/pass-var-to-closure.html" > < strong aria-hidden = "true" > 2.9.< / strong > Pass Variables to Closure< / a > < / li > < li class = "chapter-item expanded " > < a href = "../idioms/priv-extend.html" > < strong aria-hidden = "true" > 2.10.< / strong > Privacy For Extensibility< / a > < / li > < li class = "chapter-item expanded " > < a href = "../idioms/rustdoc-init.html" > < strong aria-hidden = "true" > 2.11.< / strong > Easy doc initialization< / a > < / li > < li class = "chapter-item expanded " > < a href = "../idioms/temporary-mutability.html" > < strong aria-hidden = "true" > 2.12.< / strong > Temporary mutability< / a > < / li > < / ol > < / li > < li class = "chapter-item expanded " > < a href = "../patterns/index.html" > < strong aria-hidden = "true" > 3.< / strong > Design Patterns< / a > < / li > < li > < ol class = "section" > < li class = "chapter-item expanded " > < a href = "../patterns/builder.html" > < strong aria-hidden = "true" > 3.1.< / strong > Builder< / a > < / li > < li class = "chapter-item expanded " > < a href = "../patterns/compose-structs.html" > < strong aria-hidden = "true" > 3.2.< / strong > Compose Structs< / a > < / li > < li class = "chapter-item expanded " > < a href = "../patterns/entry.html" > < strong aria-hidden = "true" > 3.3.< / strong > Entry API< / a > < / li > < li class = "chapter-item expanded " > < a href = "../patterns/fold.html" > < strong aria-hidden = "true" > 3.4.< / strong > Fold< / a > < / li > < li class = "chapter-item expanded " > < a href = "../patterns/late-bounds.html" > < strong aria-hidden = "true" > 3.5.< / strong > Late Bound Bounds< / a > < / li > < li class = "chapter-item expanded " > < a href = "../patterns/newtype.html" > < strong aria-hidden = "true" > 3.6.< / strong > Newtype< / a > < / li > < li class = "chapter-item expanded " > < a href = "../patterns/RAII.html" > < strong aria-hidden = "true" > 3.7.< / strong > RAII Guards< / a > < / li > < li class = "chapter-item expanded " > < a href = "../patterns/small-crates.html" > < strong aria-hidden = "true" > 3.8.< / strong > Prefer Small Crates< / a > < / li > < li class = "chapter-item expanded " > < a href = "../patterns/unsafe-mods.html" > < strong aria-hidden = "true" > 3.9.< / strong > Contain unsafety in small modules< / a > < / li > < li class = "chapter-item expanded " > < a href = "../patterns/visitor.html" > < strong aria-hidden = "true" > 3.10.< / strong > Visitor< / a > < / li > < / ol > < / li > < li class = "chapter-item expanded " > < a href = "../anti_patterns/index.html" > < strong aria-hidden = "true" > 4.< / strong > Anti-patterns< / a > < / li > < li > < ol class = "section" > < li class = "chapter-item expanded " > < a href = "../anti_patterns/deny-warnings.html" class = "active" > < strong aria-hidden = "true" > 4.1.< / strong > #[deny(warnings)]< / a > < / li > < li class = "chapter-item expanded " > < a href = "../anti_patterns/deref.html" > < strong aria-hidden = "true" > 4.2.< / strong > Deref Polymorphism< / a > < / li > < / ol > < / li > < li class = "chapter-item expanded " > < a href = "../fu
2021-01-01 11:28:10 +00:00
< / div >
< div id = "sidebar-resize-handle" class = "sidebar-resize-handle" > < / div >
< / nav >
< div id = "page-wrapper" class = "page-wrapper" >
< div class = "page" >
< div id = "menu-bar-hover-placeholder" > < / div >
< div id = "menu-bar" class = "menu-bar sticky bordered" >
< div class = "left-buttons" >
< button id = "sidebar-toggle" class = "icon-button" type = "button" title = "Toggle Table of Contents" aria-label = "Toggle Table of Contents" aria-controls = "sidebar" >
< i class = "fa fa-bars" > < / i >
< / button >
< button id = "theme-toggle" class = "icon-button" type = "button" title = "Change theme" aria-label = "Change theme" aria-haspopup = "true" aria-expanded = "false" aria-controls = "theme-list" >
< i class = "fa fa-paint-brush" > < / i >
< / button >
< ul id = "theme-list" class = "theme-popup" aria-label = "Themes" role = "menu" >
< li role = "none" > < button role = "menuitem" class = "theme" id = "light" > Light< / button > < / li >
< li role = "none" > < button role = "menuitem" class = "theme" id = "rust" > Rust (default)< / button > < / li >
< li role = "none" > < button role = "menuitem" class = "theme" id = "coal" > Coal< / button > < / li >
< li role = "none" > < button role = "menuitem" class = "theme" id = "navy" > Navy< / button > < / li >
< li role = "none" > < button role = "menuitem" class = "theme" id = "ayu" > Ayu< / button > < / li >
< / ul >
< button id = "search-toggle" class = "icon-button" type = "button" title = "Search. (Shortkey: s)" aria-label = "Toggle Searchbar" aria-expanded = "false" aria-keyshortcuts = "S" aria-controls = "searchbar" >
< i class = "fa fa-search" > < / i >
< / button >
< / div >
< h1 class = "menu-title" > Rust Design Patterns< / h1 >
< div class = "right-buttons" >
< a href = "../print.html" title = "Print this book" aria-label = "Print this book" >
< i id = "print-button" class = "fa fa-print" > < / i >
< / a >
< a href = "https://github.com/rust-unofficial/patterns" title = "Git repository" aria-label = "Git repository" >
< i id = "git-repository-button" class = "fa fa-github" > < / i >
< / a >
< / div >
< / div >
< div id = "search-wrapper" class = "hidden" >
< form id = "searchbar-outer" class = "searchbar-outer" >
< input type = "search" name = "search" id = "searchbar" name = "searchbar" placeholder = "Search this book ..." aria-controls = "searchresults-outer" aria-describedby = "searchresults-header" >
< / form >
< div id = "searchresults-outer" class = "searchresults-outer hidden" >
< div id = "searchresults-header" class = "searchresults-header" > < / div >
< ul id = "searchresults" >
< / ul >
< / div >
< / div >
<!-- Apply ARIA attributes after the sidebar and the sidebar toggle button are added to the DOM -->
< script type = "text/javascript" >
document.getElementById('sidebar-toggle').setAttribute('aria-expanded', sidebar === 'visible');
document.getElementById('sidebar').setAttribute('aria-hidden', sidebar !== 'visible');
Array.from(document.querySelectorAll('#sidebar a')).forEach(function(link) {
link.setAttribute('tabIndex', sidebar === 'visible' ? 0 : -1);
});
< / script >
< div id = "content" class = "content" >
< main >
< h1 > < a class = "header" href = "#denywarnings" id = "denywarnings" > < code > #![deny(warnings)]< / code > < / a > < / h1 >
< h2 > < a class = "header" href = "#description" id = "description" > Description< / a > < / h2 >
< p > A well-intentioned crate author wants to ensure their code builds without
warnings. So they annotate their crate root with the following:< / p >
< h2 > < a class = "header" href = "#example" id = "example" > Example< / a > < / h2 >
< pre > < pre class = "playground" > < code class = "language-rust edition2018" >
< span class = "boring" > #![allow(unused)]
< / span > #![deny(warnings)]
< span class = "boring" > fn main() {
< / span > // All is well.
< span class = "boring" > }
< / span > < / code > < / pre > < / pre >
< h2 > < a class = "header" href = "#advantages" id = "advantages" > Advantages< / a > < / h2 >
< p > It is short and will stop the build if anything is amiss.< / p >
< h2 > < a class = "header" href = "#drawbacks" id = "drawbacks" > Drawbacks< / a > < / h2 >
< p > By disallowing the compiler to build with warnings, a crate author opts out of
Rust's famed stability. Sometimes new features or old misfeatures need a change
in how things are done, thus lints are written that < code > warn< / code > for a certain grace
period before being turned to < code > deny< / code > .< / p >
< p > For example, it was discovered that a type could have two < code > impl< / code > s with the same
method. This was deemed a bad idea, but in order to make the transition smooth,
the < code > overlapping-inherent-impls< / code > lint was introduced to give a warning to those
stumbling on this fact, before it becomes a hard error in a future release.< / p >
< p > Also sometimes APIs get deprecated, so their use will emit a warning where
before there was none.< / p >
< p > All this conspires to potentially break the build whenever something changes.< / p >
< p > Furthermore, crates that supply additional lints (e.g. < a href = "https://github.com/Manishearth/rust-clippy" > rust-clippy< / a > ) can no
longer be used unless the annotation is removed. This is mitigated with
< a href = "https://doc.rust-lang.org/rustc/lints/levels.html#capping-lints" > --cap-lints< / a > .< / p >
< h2 > < a class = "header" href = "#alternatives" id = "alternatives" > Alternatives< / a > < / h2 >
< p > There are two ways of tackling this problem: First, we can decouple the build
setting from the code, and second, we can name the lints we want to deny
explicitly.< / p >
< p > The following command line will build with all warnings set to < code > deny< / code > :< / p >
< p > < code > RUSTFLAGS=" -D warnings" cargo build< / code > < / p >
< p > This can be done by any individual developer (or be set in a CI tool like
Travis, but remember that this may break the build when something changes)
without requiring a change to the code.< / p >
< p > Alternatively, we can specify the lints that we want to < code > deny< / code > in the code.
Here is a list of warning lints that is (hopefully) safe to deny:< / p >
2021-01-02 11:41:18 +00:00
< pre > < code class = "language-rust ignore" > #[deny(bad-style,
2021-01-01 11:28:10 +00:00
const-err,
dead-code,
extra-requirement-in-impl,
improper-ctypes,
legacy-directory-ownership,
non-shorthand-field-patterns,
no-mangle-generic-items,
overflowing-literals,
path-statements ,
patterns-in-fns-without-body,
plugin-as-library,
private-in-public,
private-no-mangle-fns,
private-no-mangle-statics,
raw-pointer-derive,
safe-extern-statics,
unconditional-recursion,
unions-with-drop-fields,
unused,
unused-allocation,
unused-comparisons,
unused-parens,
while-true)]
2021-01-02 11:41:18 +00:00
< / code > < / pre >
2021-01-01 11:28:10 +00:00
< p > In addition, the following < code > allow< / code > ed lints may be a good idea to < code > deny< / code > :< / p >
2021-01-02 11:41:18 +00:00
< pre > < code class = "language-rust ignore" > #[deny(missing-debug-implementations,
2021-01-01 11:28:10 +00:00
missing-docs,
trivial-casts,
trivial-numeric-casts,
unused-extern-crates,
unused-import-braces,
unused-qualifications,
unused-results)]
2021-01-02 11:41:18 +00:00
< / code > < / pre >
2021-01-01 11:28:10 +00:00
< p > Some may also want to add < code > missing-copy-implementations< / code > to their list.< / p >
< p > Note that we explicitly did not add the < code > deprecated< / code > lint, as it is fairly
certain that there will be more deprecated APIs in the future.< / p >
< h2 > < a class = "header" href = "#see-also" id = "see-also" > See also< / a > < / h2 >
< ul >
< li > < a href = "https://doc.rust-lang.org/reference/attributes.html#deprecation" > deprecate attribute< / a > documentation< / li >
< li > Type < code > rustc -W help< / code > for a list of lints on your system. Also type
< code > rustc --help< / code > for a general list of options< / li >
< li > < a href = "https://github.com/Manishearth/rust-clippy" > rust-clippy< / a > is a collection of lints for better Rust code< / li >
< / ul >
< / main >
< nav class = "nav-wrapper" aria-label = "Page navigation" >
<!-- Mobile navigation buttons -->
< a rel = "prev" href = "../anti_patterns/index.html" class = "mobile-nav-chapters previous" title = "Previous chapter" aria-label = "Previous chapter" aria-keyshortcuts = "Left" >
< i class = "fa fa-angle-left" > < / i >
< / a >
< a rel = "next" href = "../anti_patterns/deref.html" class = "mobile-nav-chapters next" title = "Next chapter" aria-label = "Next chapter" aria-keyshortcuts = "Right" >
< i class = "fa fa-angle-right" > < / i >
< / a >
< div style = "clear: both" > < / div >
< / nav >
< / div >
< / div >
< nav class = "nav-wide-wrapper" aria-label = "Page navigation" >
< a rel = "prev" href = "../anti_patterns/index.html" class = "nav-chapters previous" title = "Previous chapter" aria-label = "Previous chapter" aria-keyshortcuts = "Left" >
< i class = "fa fa-angle-left" > < / i >
< / a >
< a rel = "next" href = "../anti_patterns/deref.html" class = "nav-chapters next" title = "Next chapter" aria-label = "Next chapter" aria-keyshortcuts = "Right" >
< i class = "fa fa-angle-right" > < / i >
< / a >
< / nav >
< / div >
< script type = "text/javascript" >
window.playground_copyable = true;
< / script >
< script src = "../elasticlunr.min.js" type = "text/javascript" charset = "utf-8" > < / script >
< script src = "../mark.min.js" type = "text/javascript" charset = "utf-8" > < / script >
< script src = "../searcher.js" type = "text/javascript" charset = "utf-8" > < / script >
< script src = "../clipboard.min.js" type = "text/javascript" charset = "utf-8" > < / script >
< script src = "../highlight.js" type = "text/javascript" charset = "utf-8" > < / script >
< script src = "../book.js" type = "text/javascript" charset = "utf-8" > < / script >
<!-- Custom JS scripts -->
< / body >
< / html >
