Skip to content

Commit 22de929

Browse files
committed
rustdoc: Simplify desc pass
1 parent ba2c2af commit 22de929

File tree

1 file changed

+14
-60
lines changed

1 file changed

+14
-60
lines changed

src/rustdoc/desc_pass.rs

Lines changed: 14 additions & 60 deletions
Original file line numberDiff line numberDiff line change
@@ -16,14 +16,12 @@ fn run(
1616
op: op
1717
) -> doc::cratedoc {
1818
let fold = fold::fold({
19-
fold_mod: fold_mod,
20-
fold_const: fold_const,
19+
fold_item: fold_item,
2120
fold_fn: fold_fn,
2221
fold_enum: fold_enum,
2322
fold_res: fold_res,
2423
fold_iface: fold_iface,
25-
fold_impl: fold_impl,
26-
fold_type: fold_type
24+
fold_impl: fold_impl
2725
with *fold::default_seq_fold(op)
2826
});
2927
fold.fold_crate(fold, doc)
@@ -33,28 +31,12 @@ fn maybe_apply_op(op: op, s: option<str>) -> option<str> {
3331
option::map(s) {|s| op(s) }
3432
}
3533

36-
fn fold_mod(fold: fold::fold<op>, doc: doc::moddoc) -> doc::moddoc {
37-
let doc = fold::default_seq_fold_mod(fold, doc);
34+
fn fold_item(fold: fold::fold<op>, doc: doc::itemdoc) -> doc::itemdoc {
35+
let doc = fold::default_seq_fold_item(fold, doc);
3836

3937
{
40-
item: {
41-
brief: maybe_apply_op(fold.ctxt, doc.brief()),
42-
desc: maybe_apply_op(fold.ctxt, doc.desc())
43-
with doc.item
44-
}
45-
with doc
46-
}
47-
}
48-
49-
fn fold_const(fold: fold::fold<op>, doc: doc::constdoc) -> doc::constdoc {
50-
let doc = fold::default_seq_fold_const(fold, doc);
51-
52-
{
53-
item: {
54-
brief: maybe_apply_op(fold.ctxt, doc.brief()),
55-
desc: maybe_apply_op(fold.ctxt, doc.desc())
56-
with doc.item
57-
}
38+
brief: maybe_apply_op(fold.ctxt, doc.brief),
39+
desc: maybe_apply_op(fold.ctxt, doc.desc)
5840
with doc
5941
}
6042
}
@@ -63,11 +45,6 @@ fn fold_fn(fold: fold::fold<op>, doc: doc::fndoc) -> doc::fndoc {
6345
let doc = fold::default_seq_fold_fn(fold, doc);
6446

6547
{
66-
item: {
67-
brief: maybe_apply_op(fold.ctxt, doc.brief()),
68-
desc: maybe_apply_op(fold.ctxt, doc.desc())
69-
with doc.item
70-
},
7148
args: vec::map(doc.args) {|doc|
7249
{
7350
desc: maybe_apply_op(fold.ctxt, doc.desc)
@@ -84,12 +61,9 @@ fn fold_fn(fold: fold::fold<op>, doc: doc::fndoc) -> doc::fndoc {
8461
}
8562

8663
fn fold_enum(fold: fold::fold<op>, doc: doc::enumdoc) -> doc::enumdoc {
64+
let doc = fold::default_seq_fold_enum(fold, doc);
65+
8766
{
88-
item: {
89-
brief: maybe_apply_op(fold.ctxt, doc.brief()),
90-
desc: maybe_apply_op(fold.ctxt, doc.desc())
91-
with doc.item
92-
},
9367
variants: vec::map(doc.variants) {|variant|
9468
{
9569
desc: maybe_apply_op(fold.ctxt, variant.desc)
@@ -101,12 +75,9 @@ fn fold_enum(fold: fold::fold<op>, doc: doc::enumdoc) -> doc::enumdoc {
10175
}
10276

10377
fn fold_res(fold: fold::fold<op>, doc: doc::resdoc) -> doc::resdoc {
78+
let doc = fold::default_seq_fold_res(fold, doc);
79+
10480
{
105-
item: {
106-
brief: maybe_apply_op(fold.ctxt, doc.brief()),
107-
desc: maybe_apply_op(fold.ctxt, doc.desc())
108-
with doc.item
109-
},
11081
args: vec::map(doc.args) {|arg|
11182
{
11283
desc: maybe_apply_op(fold.ctxt, arg.desc)
@@ -118,12 +89,9 @@ fn fold_res(fold: fold::fold<op>, doc: doc::resdoc) -> doc::resdoc {
11889
}
11990

12091
fn fold_iface(fold: fold::fold<op>, doc: doc::ifacedoc) -> doc::ifacedoc {
92+
let doc = fold::default_seq_fold_iface(fold, doc);
93+
12194
{
122-
item: {
123-
brief: maybe_apply_op(fold.ctxt, doc.brief()),
124-
desc: maybe_apply_op(fold.ctxt, doc.desc())
125-
with doc.item
126-
},
12795
methods: apply_to_methods(fold.ctxt, doc.methods)
12896
with doc
12997
}
@@ -151,24 +119,10 @@ fn apply_to_methods(op: op, docs: [doc::methoddoc]) -> [doc::methoddoc] {
151119
}
152120

153121
fn fold_impl(fold: fold::fold<op>, doc: doc::impldoc) -> doc::impldoc {
154-
{
155-
item: {
156-
brief: maybe_apply_op(fold.ctxt, doc.brief()),
157-
desc: maybe_apply_op(fold.ctxt, doc.desc())
158-
with doc.item
159-
},
160-
methods: apply_to_methods(fold.ctxt, doc.methods)
161-
with doc
162-
}
163-
}
122+
let doc = fold::default_seq_fold_impl(fold, doc);
164123

165-
fn fold_type(fold: fold::fold<op>, doc: doc::tydoc) -> doc::tydoc {
166124
{
167-
item: {
168-
brief: maybe_apply_op(fold.ctxt, doc.brief()),
169-
desc: maybe_apply_op(fold.ctxt, doc.desc())
170-
with doc.item
171-
}
125+
methods: apply_to_methods(fold.ctxt, doc.methods)
172126
with doc
173127
}
174128
}

0 commit comments

Comments
 (0)