David Tolnay | f83b6c9 | 2019-03-09 22:58:26 -0800 | [diff] [blame] | 1 | #![cfg(not(syn_disable_nightly_tests))] |
David Tolnay | ecd024d | 2018-07-21 09:07:56 -0700 | [diff] [blame] | 2 | #![recursion_limit = "1024"] |
Nika Layzell | a2a1a4a | 2017-11-19 11:33:17 -0500 | [diff] [blame] | 3 | #![feature(rustc_private)] |
| 4 | |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 5 | //! The tests in this module do the following: |
| 6 | //! |
David Tolnay | cfa5cc0 | 2017-11-13 01:05:11 -0800 | [diff] [blame] | 7 | //! 1. Parse a given expression in both `syn` and `libsyntax`. |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 8 | //! 2. Fold over the expression adding brackets around each subexpression (with |
David Tolnay | cfa5cc0 | 2017-11-13 01:05:11 -0800 | [diff] [blame] | 9 | //! some complications - see the `syn_brackets` and `libsyntax_brackets` |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 10 | //! methods). |
| 11 | //! 3. Serialize the `syn` expression back into a string, and re-parse it with |
David Tolnay | cfa5cc0 | 2017-11-13 01:05:11 -0800 | [diff] [blame] | 12 | //! `libsyntax`. |
David Tolnay | 0ccb6d1 | 2018-08-14 22:43:00 -0700 | [diff] [blame] | 13 | //! 4. Respan all of the expressions, replacing the spans with the default |
| 14 | //! spans. |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 15 | //! 5. Compare the expressions with one another, if they are not equal fail. |
| 16 | |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 17 | extern crate quote; |
David Tolnay | ee97dbf | 2017-11-19 14:24:38 -0800 | [diff] [blame] | 18 | extern crate rayon; |
David Tolnay | d1c31cc | 2018-08-24 14:47:15 -0400 | [diff] [blame] | 19 | extern crate regex; |
David Tolnay | c865992 | 2018-08-14 22:40:50 -0700 | [diff] [blame] | 20 | extern crate rustc_data_structures; |
David Tolnay | 0eab7d9 | 2018-09-26 22:10:40 -0700 | [diff] [blame] | 21 | extern crate smallvec; |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 22 | extern crate syn; |
David Tolnay | cfa5cc0 | 2017-11-13 01:05:11 -0800 | [diff] [blame] | 23 | extern crate syntax; |
David Tolnay | fcd4a67 | 2019-01-24 20:56:54 -0800 | [diff] [blame] | 24 | extern crate syntax_pos; |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 25 | extern crate walkdir; |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 26 | |
David Tolnay | c3f9856 | 2018-11-02 08:55:05 -0700 | [diff] [blame] | 27 | mod features; |
| 28 | |
David Tolnay | e9b448f | 2019-03-10 22:46:38 -0700 | [diff] [blame] | 29 | use quote::quote; |
David Tolnay | 5138205 | 2017-12-27 13:46:21 -0500 | [diff] [blame] | 30 | use rayon::iter::{IntoParallelIterator, ParallelIterator}; |
David Tolnay | d1c31cc | 2018-08-24 14:47:15 -0400 | [diff] [blame] | 31 | use regex::Regex; |
David Tolnay | e9b448f | 2019-03-10 22:46:38 -0700 | [diff] [blame] | 32 | use smallvec::smallvec; |
David Tolnay | cfa5cc0 | 2017-11-13 01:05:11 -0800 | [diff] [blame] | 33 | use syntax::ast; |
| 34 | use syntax::ptr::P; |
David Tolnay | e6eb590 | 2019-05-23 00:00:18 -0700 | [diff] [blame] | 35 | use syntax_pos::edition::Edition; |
Igor Gnatenko | 951a52b | 2018-03-12 10:33:33 +0100 | [diff] [blame] | 36 | use walkdir::{DirEntry, WalkDir}; |
David Tolnay | ee97dbf | 2017-11-19 14:24:38 -0800 | [diff] [blame] | 37 | |
| 38 | use std::fs::File; |
| 39 | use std::io::Read; |
David Tolnay | 3eaf7d8 | 2017-12-17 23:14:52 -0800 | [diff] [blame] | 40 | use std::process; |
David Tolnay | ee97dbf | 2017-11-19 14:24:38 -0800 | [diff] [blame] | 41 | use std::sync::atomic::{AtomicUsize, Ordering}; |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 42 | |
David Tolnay | ecd024d | 2018-07-21 09:07:56 -0700 | [diff] [blame] | 43 | use common::eq::SpanlessEq; |
| 44 | use common::parse; |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 45 | |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 46 | #[macro_use] |
David Tolnay | dd12556 | 2017-12-31 02:16:22 -0500 | [diff] [blame] | 47 | mod macros; |
| 48 | |
| 49 | #[allow(dead_code)] |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 50 | mod common; |
| 51 | |
| 52 | /// Test some pre-set expressions chosen by us. |
| 53 | #[test] |
| 54 | fn test_simple_precedence() { |
| 55 | const EXPRS: &[&str] = &[ |
| 56 | "1 + 2 * 3 + 4", |
| 57 | "1 + 2 * ( 3 + 4 )", |
| 58 | "{ for i in r { } *some_ptr += 1; }", |
| 59 | "{ loop { break 5; } }", |
| 60 | "{ if true { () }.mthd() }", |
Nika Layzell | 3aa0dc7 | 2017-12-04 13:41:28 -0500 | [diff] [blame] | 61 | "{ for i in unsafe { 20 } { } }", |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 62 | ]; |
| 63 | |
| 64 | let mut failed = 0; |
| 65 | |
| 66 | for input in EXPRS { |
| 67 | let expr = if let Some(expr) = parse::syn_expr(input) { |
| 68 | expr |
| 69 | } else { |
| 70 | failed += 1; |
| 71 | continue; |
| 72 | }; |
| 73 | |
| 74 | let pf = match test_expressions(vec![expr]) { |
| 75 | (1, 0) => "passed", |
| 76 | (0, 1) => { |
| 77 | failed += 1; |
| 78 | "failed" |
| 79 | } |
| 80 | _ => unreachable!(), |
| 81 | }; |
| 82 | errorf!("=== {}: {}\n", input, pf); |
| 83 | } |
| 84 | |
| 85 | if failed > 0 { |
| 86 | panic!("Failed {} tests", failed); |
| 87 | } |
| 88 | } |
| 89 | |
| 90 | /// Test expressions from rustc, like in `test_round_trip`. |
| 91 | #[test] |
cad97 | 286b19f | 2019-01-23 23:08:25 -0500 | [diff] [blame] | 92 | #[cfg_attr(target_os = "windows", ignore = "requires nix .sh")] |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 93 | fn test_rustc_precedence() { |
Alex Crichton | 8637477 | 2017-07-07 20:39:28 -0700 | [diff] [blame] | 94 | common::clone_rust(); |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 95 | let abort_after = common::abort_after(); |
| 96 | if abort_after == 0 { |
| 97 | panic!("Skipping all precedence tests"); |
| 98 | } |
| 99 | |
David Tolnay | ee97dbf | 2017-11-19 14:24:38 -0800 | [diff] [blame] | 100 | let passed = AtomicUsize::new(0); |
| 101 | let failed = AtomicUsize::new(0); |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 102 | |
David Tolnay | d1c31cc | 2018-08-24 14:47:15 -0400 | [diff] [blame] | 103 | // 2018 edition is hard |
| 104 | let edition_regex = Regex::new(r"\b(async|try)[!(]").unwrap(); |
| 105 | |
David Tolnay | ee97dbf | 2017-11-19 14:24:38 -0800 | [diff] [blame] | 106 | WalkDir::new("tests/rust") |
Igor Gnatenko | 951a52b | 2018-03-12 10:33:33 +0100 | [diff] [blame] | 107 | .sort_by(|a, b| a.file_name().cmp(b.file_name())) |
David Tolnay | ee97dbf | 2017-11-19 14:24:38 -0800 | [diff] [blame] | 108 | .into_iter() |
| 109 | .filter_entry(common::base_dir_filter) |
| 110 | .collect::<Result<Vec<DirEntry>, walkdir::Error>>() |
| 111 | .unwrap() |
| 112 | .into_par_iter() |
David Tolnay | 5138205 | 2017-12-27 13:46:21 -0500 | [diff] [blame] | 113 | .for_each(|entry| { |
| 114 | let path = entry.path(); |
| 115 | if path.is_dir() { |
| 116 | return; |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 117 | } |
David Tolnay | 5138205 | 2017-12-27 13:46:21 -0500 | [diff] [blame] | 118 | |
| 119 | // Our version of `libsyntax` can't parse this tests |
David Tolnay | 65fb566 | 2018-05-20 20:02:28 -0700 | [diff] [blame] | 120 | if path |
| 121 | .to_str() |
David Tolnay | 5138205 | 2017-12-27 13:46:21 -0500 | [diff] [blame] | 122 | .unwrap() |
| 123 | .ends_with("optional_comma_in_match_arm.rs") |
| 124 | { |
| 125 | return; |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 126 | } |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 127 | |
David Tolnay | 5138205 | 2017-12-27 13:46:21 -0500 | [diff] [blame] | 128 | let mut file = File::open(path).unwrap(); |
| 129 | let mut content = String::new(); |
| 130 | file.read_to_string(&mut content).unwrap(); |
David Tolnay | d1c31cc | 2018-08-24 14:47:15 -0400 | [diff] [blame] | 131 | let content = edition_regex.replace_all(&content, "_$0"); |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 132 | |
David Tolnay | 5138205 | 2017-12-27 13:46:21 -0500 | [diff] [blame] | 133 | let (l_passed, l_failed) = match syn::parse_file(&content) { |
| 134 | Ok(file) => { |
| 135 | let exprs = collect_exprs(file); |
| 136 | test_expressions(exprs) |
| 137 | } |
| 138 | Err(msg) => { |
| 139 | errorf!("syn failed to parse\n{:?}\n", msg); |
| 140 | (0, 1) |
| 141 | } |
| 142 | }; |
David Tolnay | ee97dbf | 2017-11-19 14:24:38 -0800 | [diff] [blame] | 143 | |
David Tolnay | 5138205 | 2017-12-27 13:46:21 -0500 | [diff] [blame] | 144 | errorf!( |
| 145 | "=== {}: {} passed | {} failed\n", |
| 146 | path.display(), |
| 147 | l_passed, |
| 148 | l_failed |
| 149 | ); |
| 150 | |
| 151 | passed.fetch_add(l_passed, Ordering::SeqCst); |
| 152 | let prev_failed = failed.fetch_add(l_failed, Ordering::SeqCst); |
| 153 | |
| 154 | if prev_failed + l_failed >= abort_after { |
| 155 | process::exit(1); |
| 156 | } |
| 157 | }); |
David Tolnay | ee97dbf | 2017-11-19 14:24:38 -0800 | [diff] [blame] | 158 | |
| 159 | let passed = passed.load(Ordering::SeqCst); |
| 160 | let failed = failed.load(Ordering::SeqCst); |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 161 | |
| 162 | errorf!("\n===== Precedence Test Results =====\n"); |
| 163 | errorf!("{} passed | {} failed\n", passed, failed); |
| 164 | |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 165 | if failed > 0 { |
| 166 | panic!("{} failures", failed); |
| 167 | } |
| 168 | } |
| 169 | |
David Tolnay | ee97dbf | 2017-11-19 14:24:38 -0800 | [diff] [blame] | 170 | fn test_expressions(exprs: Vec<syn::Expr>) -> (usize, usize) { |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 171 | let mut passed = 0; |
| 172 | let mut failed = 0; |
| 173 | |
David Tolnay | e6eb590 | 2019-05-23 00:00:18 -0700 | [diff] [blame] | 174 | syntax::with_globals(Edition::Edition2018, || { |
David Tolnay | eb7d79b | 2018-03-31 22:52:17 +0200 | [diff] [blame] | 175 | for expr in exprs { |
| 176 | let raw = quote!(#expr).to_string(); |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 177 | |
David Tolnay | eb7d79b | 2018-03-31 22:52:17 +0200 | [diff] [blame] | 178 | let libsyntax_ast = if let Some(e) = libsyntax_parse_and_rewrite(&raw) { |
| 179 | e |
| 180 | } else { |
| 181 | failed += 1; |
| 182 | errorf!("\nFAIL - libsyntax failed to parse raw\n"); |
| 183 | continue; |
| 184 | }; |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 185 | |
David Tolnay | eb7d79b | 2018-03-31 22:52:17 +0200 | [diff] [blame] | 186 | let syn_expr = syn_brackets(expr); |
| 187 | let syn_ast = if let Some(e) = parse::libsyntax_expr("e!(#syn_expr).to_string()) { |
| 188 | e |
| 189 | } else { |
| 190 | failed += 1; |
| 191 | errorf!("\nFAIL - libsyntax failed to parse bracketed\n"); |
| 192 | continue; |
| 193 | }; |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 194 | |
David Tolnay | ecd024d | 2018-07-21 09:07:56 -0700 | [diff] [blame] | 195 | if SpanlessEq::eq(&syn_ast, &libsyntax_ast) { |
David Tolnay | eb7d79b | 2018-03-31 22:52:17 +0200 | [diff] [blame] | 196 | passed += 1; |
| 197 | } else { |
| 198 | failed += 1; |
| 199 | errorf!("\nFAIL\n{:?}\n!=\n{:?}\n", syn_ast, libsyntax_ast); |
| 200 | } |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 201 | } |
David Tolnay | 5d1a3ee | 2018-03-17 19:41:36 -0700 | [diff] [blame] | 202 | }); |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 203 | |
| 204 | (passed, failed) |
| 205 | } |
| 206 | |
David Tolnay | cfa5cc0 | 2017-11-13 01:05:11 -0800 | [diff] [blame] | 207 | fn libsyntax_parse_and_rewrite(input: &str) -> Option<P<ast::Expr>> { |
David Tolnay | 3cede94 | 2017-12-26 12:29:24 -0500 | [diff] [blame] | 208 | parse::libsyntax_expr(input).and_then(libsyntax_brackets) |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 209 | } |
| 210 | |
| 211 | /// Wrap every expression which is not already wrapped in parens with parens, to |
David Tolnay | 0ccb6d1 | 2018-08-14 22:43:00 -0700 | [diff] [blame] | 212 | /// reveal the precidence of the parsed expressions, and produce a stringified |
| 213 | /// form of the resulting expression. |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 214 | /// |
David Tolnay | cfa5cc0 | 2017-11-13 01:05:11 -0800 | [diff] [blame] | 215 | /// This method operates on libsyntax objects. |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 216 | fn libsyntax_brackets(mut libsyntax_expr: P<ast::Expr>) -> Option<P<ast::Expr>> { |
David Tolnay | c865992 | 2018-08-14 22:40:50 -0700 | [diff] [blame] | 217 | use rustc_data_structures::thin_vec::ThinVec; |
David Tolnay | 0eab7d9 | 2018-09-26 22:10:40 -0700 | [diff] [blame] | 218 | use smallvec::SmallVec; |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 219 | use std::mem; |
David Tolnay | e6b3709 | 2019-05-08 18:29:54 -0700 | [diff] [blame] | 220 | use syntax::ast::{AwaitOrigin, Expr, ExprKind, Field, Mac, Pat, Stmt, StmtKind, Ty}; |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 221 | use syntax::mut_visit::{self, MutVisitor}; |
David Tolnay | fcd4a67 | 2019-01-24 20:56:54 -0800 | [diff] [blame] | 222 | use syntax_pos::DUMMY_SP; |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 223 | |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 224 | struct BracketsVisitor { |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 225 | failed: bool, |
| 226 | }; |
David Tolnay | e6b3709 | 2019-05-08 18:29:54 -0700 | [diff] [blame] | 227 | |
| 228 | impl BracketsVisitor { |
| 229 | fn recurse_expr(&mut self, e: &mut Expr) { |
| 230 | match e.node { |
| 231 | ExprKind::Await(AwaitOrigin::MacroLike, _) => { |
| 232 | // Syn sees await!() as macro and doesn't recurse inside, so |
| 233 | // skip it here too. |
| 234 | } |
| 235 | _ => mut_visit::noop_visit_expr(e, self), |
| 236 | } |
| 237 | } |
| 238 | } |
| 239 | |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 240 | impl MutVisitor for BracketsVisitor { |
| 241 | fn visit_expr(&mut self, e: &mut P<Expr>) { |
David Tolnay | e6b3709 | 2019-05-08 18:29:54 -0700 | [diff] [blame] | 242 | self.recurse_expr(e); |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 243 | match e.node { |
David Tolnay | c992d70 | 2019-06-23 18:48:23 -0700 | [diff] [blame] | 244 | ExprKind::If(..) | ExprKind::Block(..) | ExprKind::Let(..) => {} |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 245 | _ => { |
| 246 | let inner = mem::replace( |
| 247 | e, |
| 248 | P(Expr { |
| 249 | id: ast::DUMMY_NODE_ID, |
| 250 | node: ExprKind::Err, |
| 251 | span: DUMMY_SP, |
| 252 | attrs: ThinVec::new(), |
| 253 | }), |
| 254 | ); |
| 255 | e.node = ExprKind::Paren(inner); |
David Tolnay | 5d314dc | 2018-07-21 16:40:01 -0700 | [diff] [blame] | 256 | } |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 257 | } |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 258 | } |
| 259 | |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 260 | fn visit_field(&mut self, f: &mut Field) { |
| 261 | if f.is_shorthand { |
David Tolnay | e6b3709 | 2019-05-08 18:29:54 -0700 | [diff] [blame] | 262 | self.recurse_expr(&mut f.expr); |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 263 | } else { |
| 264 | self.visit_expr(&mut f.expr); |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 265 | } |
| 266 | } |
| 267 | |
| 268 | // We don't want to look at expressions that might appear in patterns or |
| 269 | // types yet. We'll look into comparing those in the future. For now |
| 270 | // focus on expressions appearing in other places. |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 271 | fn visit_pat(&mut self, pat: &mut P<Pat>) { |
| 272 | let _ = pat; |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 273 | } |
| 274 | |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 275 | fn visit_ty(&mut self, ty: &mut P<Ty>) { |
| 276 | let _ = ty; |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 277 | } |
| 278 | |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 279 | fn flat_map_stmt(&mut self, stmt: Stmt) -> SmallVec<[Stmt; 1]> { |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 280 | let node = match stmt.node { |
| 281 | // Don't wrap toplevel expressions in statements. |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 282 | StmtKind::Expr(mut e) => { |
David Tolnay | e6b3709 | 2019-05-08 18:29:54 -0700 | [diff] [blame] | 283 | self.recurse_expr(&mut e); |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 284 | StmtKind::Expr(e) |
| 285 | } |
| 286 | StmtKind::Semi(mut e) => { |
David Tolnay | e6b3709 | 2019-05-08 18:29:54 -0700 | [diff] [blame] | 287 | self.recurse_expr(&mut e); |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 288 | StmtKind::Semi(e) |
| 289 | } |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 290 | s => s, |
| 291 | }; |
| 292 | |
David Tolnay | 0eab7d9 | 2018-09-26 22:10:40 -0700 | [diff] [blame] | 293 | smallvec![Stmt { node, ..stmt }] |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 294 | } |
| 295 | |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 296 | fn visit_mac(&mut self, mac: &mut Mac) { |
David Tolnay | cfa5cc0 | 2017-11-13 01:05:11 -0800 | [diff] [blame] | 297 | // By default when folding over macros, libsyntax panics. This is |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 298 | // because it's usually not what you want, you want to run after |
| 299 | // macro expansion. We do want to do that (syn doesn't do macro |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 300 | // expansion), so we implement visit_mac to just return the macro |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 301 | // unchanged. |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 302 | let _ = mac; |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 303 | } |
| 304 | } |
| 305 | |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 306 | let mut folder = BracketsVisitor { failed: false }; |
| 307 | folder.visit_expr(&mut libsyntax_expr); |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 308 | if folder.failed { |
| 309 | None |
| 310 | } else { |
David Tolnay | 176838f | 2019-02-07 17:28:37 +0100 | [diff] [blame] | 311 | Some(libsyntax_expr) |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 312 | } |
| 313 | } |
| 314 | |
| 315 | /// Wrap every expression which is not already wrapped in parens with parens, to |
David Tolnay | 0ccb6d1 | 2018-08-14 22:43:00 -0700 | [diff] [blame] | 316 | /// reveal the precedence of the parsed expressions, and produce a stringified |
| 317 | /// form of the resulting expression. |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 318 | fn syn_brackets(syn_expr: syn::Expr) -> syn::Expr { |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 319 | use syn::fold::*; |
David Tolnay | eb7d79b | 2018-03-31 22:52:17 +0200 | [diff] [blame] | 320 | use syn::*; |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 321 | |
David Tolnay | eb75206 | 2018-01-06 13:51:42 -0800 | [diff] [blame] | 322 | struct ParenthesizeEveryExpr; |
| 323 | impl Fold for ParenthesizeEveryExpr { |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 324 | fn fold_expr(&mut self, expr: Expr) -> Expr { |
David Tolnay | 8c91b88 | 2017-12-28 23:04:32 -0500 | [diff] [blame] | 325 | match expr { |
| 326 | Expr::Group(_) => unreachable!(), |
David Tolnay | 9c11912 | 2018-09-01 18:47:02 -0700 | [diff] [blame] | 327 | Expr::If(..) | Expr::Unsafe(..) | Expr::Block(..) | Expr::Let(..) => { |
David Tolnay | 3bc597f | 2017-12-31 02:31:11 -0500 | [diff] [blame] | 328 | fold_expr(self, expr) |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 329 | } |
David Tolnay | 5d314dc | 2018-07-21 16:40:01 -0700 | [diff] [blame] | 330 | node => Expr::Paren(ExprParen { |
| 331 | attrs: Vec::new(), |
| 332 | expr: Box::new(fold_expr(self, node)), |
| 333 | paren_token: token::Paren::default(), |
| 334 | }), |
David Tolnay | 8c91b88 | 2017-12-28 23:04:32 -0500 | [diff] [blame] | 335 | } |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 336 | } |
| 337 | |
| 338 | fn fold_stmt(&mut self, stmt: Stmt) -> Stmt { |
| 339 | match stmt { |
| 340 | // Don't wrap toplevel expressions in statements. |
David Tolnay | 1f0b7b8 | 2018-01-06 16:07:14 -0800 | [diff] [blame] | 341 | Stmt::Expr(e) => Stmt::Expr(fold_expr(self, e)), |
| 342 | Stmt::Semi(e, semi) => Stmt::Semi(fold_expr(self, e), semi), |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 343 | s => s, |
| 344 | } |
| 345 | } |
| 346 | |
| 347 | // We don't want to look at expressions that might appear in patterns or |
| 348 | // types yet. We'll look into comparing those in the future. For now |
| 349 | // focus on expressions appearing in other places. |
| 350 | fn fold_pat(&mut self, pat: Pat) -> Pat { |
| 351 | pat |
| 352 | } |
| 353 | |
David Tolnay | fd6bf5c | 2017-11-12 09:41:14 -0800 | [diff] [blame] | 354 | fn fold_type(&mut self, ty: Type) -> Type { |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 355 | ty |
| 356 | } |
| 357 | } |
| 358 | |
David Tolnay | eb75206 | 2018-01-06 13:51:42 -0800 | [diff] [blame] | 359 | let mut folder = ParenthesizeEveryExpr; |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 360 | folder.fold_expr(syn_expr) |
| 361 | } |
| 362 | |
| 363 | /// Walk through a crate collecting all expressions we can find in it. |
David Tolnay | c7a5d3d | 2017-06-04 12:11:05 -0700 | [diff] [blame] | 364 | fn collect_exprs(file: syn::File) -> Vec<syn::Expr> { |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 365 | use syn::fold::*; |
David Tolnay | f2cfd72 | 2017-12-31 18:02:51 -0500 | [diff] [blame] | 366 | use syn::punctuated::Punctuated; |
David Tolnay | eb7d79b | 2018-03-31 22:52:17 +0200 | [diff] [blame] | 367 | use syn::*; |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 368 | |
David Tolnay | eb75206 | 2018-01-06 13:51:42 -0800 | [diff] [blame] | 369 | struct CollectExprs(Vec<Expr>); |
| 370 | impl Fold for CollectExprs { |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 371 | fn fold_expr(&mut self, expr: Expr) -> Expr { |
| 372 | self.0.push(expr); |
| 373 | |
David Tolnay | 8c91b88 | 2017-12-28 23:04:32 -0500 | [diff] [blame] | 374 | Expr::Tuple(ExprTuple { |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 375 | attrs: vec![], |
David Tolnay | f2cfd72 | 2017-12-31 18:02:51 -0500 | [diff] [blame] | 376 | elems: Punctuated::new(), |
David Tolnay | 8c91b88 | 2017-12-28 23:04:32 -0500 | [diff] [blame] | 377 | paren_token: token::Paren::default(), |
| 378 | }) |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 379 | } |
| 380 | } |
| 381 | |
David Tolnay | eb75206 | 2018-01-06 13:51:42 -0800 | [diff] [blame] | 382 | let mut folder = CollectExprs(vec![]); |
David Tolnay | c7a5d3d | 2017-06-04 12:11:05 -0700 | [diff] [blame] | 383 | folder.fold_file(file); |
Michael Layzell | 53fc31a | 2017-06-07 09:21:53 -0400 | [diff] [blame] | 384 | folder.0 |
| 385 | } |