From 4dbf71d33291ab9c6d83a06fa88f322d7078034c Mon Sep 17 00:00:00 2001 From: DJMcNab <36049421+DJMcNab@users.noreply.github.com> Date: Fri, 21 Dec 2018 22:41:09 +0000 Subject: Remove uneeded todo in lexer --- crates/ra_syntax/src/lexer.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/crates/ra_syntax/src/lexer.rs b/crates/ra_syntax/src/lexer.rs index f388da273..c6acd095e 100644 --- a/crates/ra_syntax/src/lexer.rs +++ b/crates/ra_syntax/src/lexer.rs @@ -160,7 +160,7 @@ fn next_token_inner(c: char, ptr: &mut Ptr) -> SyntaxKind { // if we find one, then this is an invalid character literal if ptr.at('\'') { ptr.bump(); - return CHAR; // TODO: error reporting + return CHAR; } LIFETIME } else { -- cgit v1.2.3 From 1dffbe5c379894add3097c005a8b0e4c70173e4a Mon Sep 17 00:00:00 2001 From: DJMcNab <36049421+DJMcNab@users.noreply.github.com> Date: Fri, 21 Dec 2018 22:44:31 +0000 Subject: Remove TODO on self_hosting_parsing This improving this code is not a good use of people-time, and this might be the most performant approach nonwithstanding an api for this use case being added to walkdir --- crates/ra_syntax/tests/test.rs | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/crates/ra_syntax/tests/test.rs b/crates/ra_syntax/tests/test.rs index 14ad836b5..4266864bd 100644 --- a/crates/ra_syntax/tests/test.rs +++ b/crates/ra_syntax/tests/test.rs @@ -65,14 +65,10 @@ fn self_hosting_parsing() { for entry in walkdir::WalkDir::new(dir) .into_iter() .filter_entry(|entry| { - !entry - .path() - .components() - // TODO: this more neatly - .any(|component| { - // Get all files which are not in the crates/ra_syntax/tests/data folder - component == Component::Normal(OsStr::new("data")) - }) + !entry.path().components().any(|component| { + // Get all files which are not in the crates/ra_syntax/tests/data folder + component == Component::Normal(OsStr::new("data")) + }) }) .map(|e| e.unwrap()) .filter(|entry| { -- cgit v1.2.3