Allow multiple indented blocks in REPL

Jack O'Connor · Jack O'Connor · commit 135ccae33d82 · 2025-08-15T23:05:10.000+01:00
There aren't any tests but being able to do nested for loops seems like
a pretty big win to me so I'm going to put up for review.

The original returned boolean clearly had **false positives** for
detecting bad errors for things like nested `if` and `for` statements.
What is less clear is if there are any **true positives** which I am no
longer catching with the updated return value.
diff --git a/src/shell.rs b/src/shell.rs
@@ -86,11 +86,7 @@ fn shell_exec(
                             continuing_block
                         } // && p.location.is_some()
                         ParseErrorType::OtherError(msg) => {
-                            if msg.starts_with("Expected an indented block") {
-                                continuing_block
-                            } else {
-                                true
-                            }
+                            !msg.starts_with("Expected an indented block")
                         }
                         _ => true, // !matches!(p, ParseErrorType::UnrecognizedToken(Tok::Dedent, _))
                     }

Original file line number	Diff line number	Diff line change
`@@ -86,11 +86,7 @@ fn shell_exec(`
`86`	`86`	`continuing_block`
`87`	`87`	`} // && p.location.is_some()`
`88`	`88`	`ParseErrorType::OtherError(msg) => {`
`89`		`- if msg.starts_with("Expected an indented block") {`
`90`		`- continuing_block`
`91`		`- } else {`
`92`		`- true`
`93`		`- }`
	`89`	`+ !msg.starts_with("Expected an indented block")`
`94`	`90`	`}`
`95`	`91`	`_ => true, // !matches!(p, ParseErrorType::UnrecognizedToken(Tok::Dedent, _))`
`96`	`92`	`}`