Move "Expected indented block" error message's line number back to where the error is

[miv391]

Fixes #81046

In master "Expected indented block ..." error messages are assigned to code lines likes this:

The error message at line 7 tells that "Expected indented block after class declaration". The class declaration mentioned in error messages in not at line 7, it is at line 4. Other errors in screenshot are similar. The most confusing (I have seen this at Reddit discussions several times) is the error at line 16 which mentions function declaration. But the problem is not the function declaration at line 16, the problematic function declaration is at line 11.

In master, the error message line is obviously the line where the missing indented block is noticed.

This PR fixes this problem and now the same errors look like this:

Now all "Expected indented block ..." error messages are pointing to lines which are actually expecting something.

Other kind of errors seem to be unaffected by this change:

But I haven't tested with every different error message and I don't have deep understanding about GDScript parser, so something may have broken. If there exists somewhere a test project which contains all possible GDScript error, I would be interested about that.

It might also be worth considering about changing these "Expected indented block ..." error messages a bit.

"Expected indented block after class declaration"
-->
"Expected indented block after this class declaration"

Adding "this" would emphasize the fact that it is exactly this function that needs indented block after it. But I may be overcautious here.

[dalexeev]

Makes sense to me since the current token can be separated from previous by empty lines or comments. I rebased and tested it locally, looks correct.

It would be nice to add some tests for the parser to prevent regressions:

800: 74: 	if (multiline && !consume(GDScriptTokenizer::Token::INDENT, R"(Expected indented block after class declaration.)")) {
1105: 62: 		if (!consume(GDScriptTokenizer::Token::INDENT, R"(Expected indented block for property after ":".)")) {
1127: 56: 			push_error("Property with inline code must go to an indented block.");
1180: 64: 		consume(GDScriptTokenizer::Token::DEDENT, R"(Expected end of indented block for property.)");
1679: 70: 		if (!consume(GDScriptTokenizer::Token::INDENT, vformat(R"(Expected indented block after %s.)", p_context))) {
2099: 64: 	if (!consume(GDScriptTokenizer::Token::INDENT, R"(Expected an indented block after "match" statement.)")) {
2151: 59: 	consume(GDScriptTokenizer::Token::DEDENT, R"(Expected an indented block after "match" statement.)");

[AThousandShips]

Should probably also resolve:

• Wrong error line indicator position of Static typed for loop #92681

[dalexeev]

Should probably also resolve:

• Wrong error line indicator position of Static typed for loop #92681

No, I checked. #92681 is about a runtime error. It looks like it's being handled elsewhere, I'll take a look later.

[akien-mga]

Thanks!