rustdoc: deny(deprecated_mode)

Author: brson

src/librustdoc/astsrv.rs

@@ -62,19 +62,19 @@ impl Srv: Clone {
     fn clone(&self) -> Srv { copy *self }
 }
 
-pub fn from_str<T>(source: ~str, owner: SrvOwner<T>) -> T {
+pub fn from_str<T>(+source: ~str, owner: SrvOwner<T>) -> T {
     run(owner, copy source, parse::from_str_sess)
 }
 
-pub fn from_file<T>(file: ~str, owner: SrvOwner<T>) -> T {
+pub fn from_file<T>(+file: ~str, owner: SrvOwner<T>) -> T {
     run(owner, copy file, |sess, f| parse::from_file_sess(sess, &Path(f)))
 }
 
-fn run<T>(owner: SrvOwner<T>, source: ~str, +parse: Parser) -> T {
+fn run<T>(owner: SrvOwner<T>, +source: ~str, +parse: Parser) -> T {
 
     let srv_ = Srv({
         ch: do util::spawn_listener |copy source, move parse, po| {
-            act(po, copy source, parse);
+            act(po, copy source, copy parse);
         }
     });
 
@@ -83,7 +83,7 @@ fn run<T>(owner: SrvOwner<T>, source: ~str, +parse: Parser) -> T {
     move res
 }
 
-fn act(po: oldcomm::Port<Msg>, source: ~str, parse: Parser) {
+fn act(po: oldcomm::Port<Msg>, +source: ~str, +parse: Parser) {
     let sess = build_session();
 
     let ctxt = build_ctxt(

src/librustdoc/attr_pass.rs

@@ -221,7 +221,7 @@ fn fold_trait(
 fn merge_method_attrs(
     srv: astsrv::Srv,
     item_id: doc::AstId,
-    docs: ~[doc::MethodDoc]
+    +docs: ~[doc::MethodDoc]
 ) -> ~[doc::MethodDoc] {
 
     // Create an assoc list from method name to attributes
@@ -319,7 +319,7 @@ mod test {
     use doc;
     use extract;
 
-    pub fn mk_doc(source: ~str) -> doc::Doc {
+    pub fn mk_doc(+source: ~str) -> doc::Doc {
         do astsrv::from_str(copy source) |srv| {
             let doc = extract::from_srv(srv, ~"");
             run(srv, doc)

src/librustdoc/demo.rs

@@ -39,8 +39,8 @@ enum OmNomNomy {
 }
 
 fn take_my_order_please(
-    _waitperson: WaitPerson,
-    _order: ~[OmNomNomy]
+    +_waitperson: WaitPerson,
+    +_order: ~[OmNomNomy]
 ) -> uint {
 
     /*!

src/librustdoc/desc_to_brief_pass.rs

@@ -52,7 +52,7 @@ fn fold_item(fold: &fold::Fold<()>, +doc: doc::ItemDoc) -> doc::ItemDoc {
     let doc = fold::default_seq_fold_item(fold, doc);
 
     doc::ItemDoc {
-        brief: extract(doc.desc),
+        brief: extract(copy doc.desc),
         .. doc
     }
 }
@@ -62,7 +62,7 @@ fn fold_trait(fold: &fold::Fold<()>, +doc: doc::TraitDoc) -> doc::TraitDoc {
 
     doc::TraitDoc {
         methods: par::map(doc.methods, |doc| doc::MethodDoc {
-            brief: extract(doc.desc),
+            brief: extract(copy doc.desc),
             .. copy *doc
         }),
         .. doc
@@ -74,7 +74,7 @@ fn fold_impl(fold: &fold::Fold<()>, +doc: doc::ImplDoc) -> doc::ImplDoc {
 
     doc::ImplDoc {
         methods: par::map(doc.methods, |doc| doc::MethodDoc {
-            brief: extract(doc.desc),
+            brief: extract(copy doc.desc),
             .. copy *doc
         }),
         .. doc
@@ -108,7 +108,7 @@ pub mod test {
     use doc;
     use extract;
 
-    pub fn mk_doc(source: ~str) -> doc::Doc {
+    pub fn mk_doc(+source: ~str) -> doc::Doc {
         do astsrv::from_str(copy source) |srv| {
             let doc = extract::from_srv(srv, ~"");
             let doc = (attr_pass::mk_pass().f)(srv, doc);
@@ -117,15 +117,15 @@ pub mod test {
     }
 }
 
-fn extract(desc: Option<~str>) -> Option<~str> {
+fn extract(+desc: Option<~str>) -> Option<~str> {
     if desc.is_none() {
         return None
     }
 
     parse_desc((copy desc).get())
 }
 
-fn parse_desc(desc: ~str) -> Option<~str> {
+fn parse_desc(+desc: ~str) -> Option<~str> {
 
     const max_brief_len: uint = 120u;
 
@@ -141,7 +141,7 @@ fn parse_desc(desc: ~str) -> Option<~str> {
     }
 }
 
-fn first_sentence(s: ~str) -> Option<~str> {
+fn first_sentence(+s: ~str) -> Option<~str> {
     let paras = paragraphs(copy s);
     if !paras.is_empty() {
         let first_para = vec::head(paras);
@@ -151,7 +151,7 @@ fn first_sentence(s: ~str) -> Option<~str> {
     }
 }
 
-fn first_sentence_(s: ~str) -> ~str {
+fn first_sentence_(+s: ~str) -> ~str {
     let mut dotcount = 0;
     // The index of the character following a single dot. This allows
     // Things like [0..1) to appear in the brief description
@@ -182,7 +182,7 @@ fn first_sentence_(s: ~str) -> ~str {
     }
 }
 
-fn paragraphs(s: ~str) -> ~[~str] {
+fn paragraphs(+s: ~str) -> ~[~str] {
     let lines = str::lines_any(s);
     let mut whitespace_lines = 0;
     let mut accum = ~"";
@@ -233,7 +233,7 @@ fn test_paragraphs_2() {
 #[test]
 fn should_promote_short_descs() {
     let desc = Some(~"desc");
-    let brief = extract(desc);
+    let brief = extract(copy desc);
     assert brief == desc;
 }
 

src/librustdoc/escape_pass.rs

@@ -19,7 +19,7 @@ pub fn mk_pass() -> Pass {
     text_pass::mk_pass(~"escape", escape)
 }
 
-fn escape(s: ~str) -> ~str {
+fn escape(s: &str) -> ~str {
     str::replace(s, ~"\\", ~"\\\\")
 }
 

src/librustdoc/extract.rs

@@ -71,7 +71,7 @@ fn top_moddoc_from_crate(
     +default_name: ~str
 ) -> doc::ModDoc {
     moddoc_from_mod(mk_itemdoc(ast::crate_node_id, default_name),
-                    crate.node.module)
+                    copy crate.node.module)
 }
 
 fn mk_itemdoc(id: ast::node_id, +name: ~str) -> doc::ItemDoc {
@@ -88,7 +88,7 @@ fn mk_itemdoc(id: ast::node_id, +name: ~str) -> doc::ItemDoc {
 
 fn moddoc_from_mod(
     +itemdoc: doc::ItemDoc,
-    module_: ast::_mod
+    +module_: ast::_mod
 ) -> doc::ModDoc {
     doc::ModDoc {
         item: itemdoc,
@@ -149,7 +149,7 @@ fn moddoc_from_mod(
 
 fn nmoddoc_from_mod(
     +itemdoc: doc::ItemDoc,
-    module_: ast::foreign_mod
+    +module_: ast::foreign_mod
 ) -> doc::NmodDoc {
     let mut fns = ~[];
     for module_.items.each |item| {
@@ -273,7 +273,7 @@ fn should_extract_trait_methods() {
 
 fn impldoc_from_impl(
     +itemdoc: doc::ItemDoc,
-    methods: ~[@ast::method]
+    +methods: ~[@ast::method]
 ) -> doc::ImplDoc {
     doc::ImplDoc {
         item: itemdoc,

src/librustdoc/page_pass.rs

@@ -81,7 +81,7 @@ fn make_doc_from_pages(page_port: PagePort) -> doc::Doc {
     }
 }
 
-fn find_pages(doc: doc::Doc, page_chan: PageChan) {
+fn find_pages(+doc: doc::Doc, page_chan: PageChan) {
     let fold = Fold {
         fold_crate: fold_crate,
         fold_mod: fold_mod,
@@ -101,7 +101,7 @@ fn fold_crate(
     let doc = fold::default_seq_fold_crate(fold, doc);
 
     let page = doc::CratePage(doc::CrateDoc {
-        topmod: strip_mod(doc.topmod),
+        topmod: strip_mod(copy doc.topmod),
         .. copy doc
     });
 
@@ -119,15 +119,15 @@ fn fold_mod(
 
     if doc.id() != ast::crate_node_id {
 
-        let doc = strip_mod(doc);
+        let doc = strip_mod(copy doc);
         let page = doc::ItemPage(doc::ModTag(doc));
         oldcomm::send(fold.ctxt.op, Some(page));
     }
 
     doc
 }
 
-fn strip_mod(doc: doc::ModDoc) -> doc::ModDoc {
+fn strip_mod(+doc: doc::ModDoc) -> doc::ModDoc {
     doc::ModDoc {
         items: do doc.items.filtered |item| {
             match *item {
@@ -193,15 +193,15 @@ mod test {
 
     pub fn mk_doc_(
         output_style: config::OutputStyle,
-        source: ~str
+        +source: ~str
     ) -> doc::Doc {
         do astsrv::from_str(copy source) |srv| {
             let doc = extract::from_srv(srv, ~"");
             run(srv, doc, output_style)
         }
     }
 
-    pub fn mk_doc(source: ~str) -> doc::Doc {
+    pub fn mk_doc(+source: ~str) -> doc::Doc {
         mk_doc_(config::DocPerMod, copy source)
     }
 }

src/librustdoc/prune_hidden_pass.rs

@@ -50,7 +50,7 @@ fn fold_mod(
     }
 }
 
-fn is_hidden(srv: astsrv::Srv, doc: doc::ItemDoc) -> bool {
+fn is_hidden(srv: astsrv::Srv, +doc: doc::ItemDoc) -> bool {
     use syntax::ast_map;
 
     let id = doc.id;
@@ -76,7 +76,7 @@ pub mod test {
     use extract;
     use prune_hidden_pass::run;
 
-    pub fn mk_doc(source: ~str) -> doc::Doc {
+    pub fn mk_doc(+source: ~str) -> doc::Doc {
         do astsrv::from_str(copy source) |srv| {
             let doc = extract::from_srv(srv, ~"");
             run(srv, doc)

src/librustdoc/prune_private_pass.rs

@@ -51,7 +51,7 @@ fn fold_mod(
     }
 }
 
-fn is_visible(srv: astsrv::Srv, doc: doc::ItemDoc) -> bool {
+fn is_visible(srv: astsrv::Srv, +doc: doc::ItemDoc) -> bool {
     use syntax::ast_map;
     use syntax::ast;
 
@@ -80,7 +80,7 @@ pub mod test {
     use extract;
     use prune_private_pass::run;
 
-    pub fn mk_doc(source: ~str) -> doc::Doc {
+    pub fn mk_doc(+source: ~str) -> doc::Doc {
         do astsrv::from_str(copy source) |srv| {
             let doc = extract::from_srv(srv, ~"");
             run(srv, doc)

src/librustdoc/rustdoc.rc

@@ -24,7 +24,6 @@
 #[legacy_records];
 
 #[allow(non_implicitly_copyable_typarams)];
-#[allow(deprecated_mode)];
 #[allow(deprecated_self)];
 
 extern mod core(vers = "0.6");
@@ -89,7 +88,7 @@ fn main() {
 }
 
 /// Runs rustdoc over the given file
-fn run(config: Config) {
+fn run(+config: Config) {
 
     let source_file = copy config.input_crate;
 
@@ -148,7 +147,7 @@ fn run(config: Config) {
     }
 }
 
-fn time<T>(what: ~str, f: fn() -> T) -> T {
+fn time<T>(+what: ~str, f: fn() -> T) -> T {
     let start = std::time::precise_time_s();
     let rv = f();
     let end = std::time::precise_time_s();