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