Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multi-line string interpolation block #203

Closed
Gama11 opened this issue Aug 18, 2018 · 1 comment · Fixed by #264 or #265
Closed

Multi-line string interpolation block #203

Gama11 opened this issue Aug 18, 2018 · 1 comment · Fixed by #264 or #265
Labels

Comments

@Gama11
Copy link
Member

Gama11 commented Aug 18, 2018

class Main {
	public static function main() {
		trace('
			${
				"foo"
			}
		');
	}
}
failed to create parser context failed to make tokens Unexpected

Called from formatter/codedata/ParsedCode.hx line 38
Called from formatter/codedata/ParsedCode.hx line 6
Called from formatter/Formatter.hx line 40

Works fine if ${"foo"} is single-line.

@Gama11 Gama11 added the crash label Aug 18, 2018
AlexHaxe added a commit to AlexHaxe/haxe-formatter that referenced this issue Nov 6, 2018
AlexHaxe added a commit that referenced this issue Nov 6, 2018
* added testcase for multiline string interpolation, fixes #261
* should fix #203 too, but fails on Haxe 4 preview 5
@AlexHaxe
Copy link
Member

AlexHaxe commented Nov 6, 2018

fails on Haxe 4 preview 5 with Invalid_argument("index out of bounds")

see HaxeFoundation/haxe#7601

@AlexHaxe AlexHaxe reopened this Nov 6, 2018
AlexHaxe added a commit to AlexHaxe/haxe-formatter that referenced this issue Nov 6, 2018
use stderr for error messages, fixes HaxeCheckstyle#262
fixed index out of bound, fixes HaxeCheckstyle#203
AlexHaxe added a commit that referenced this issue Nov 6, 2018
* fixed metadata whitespace, fixes #263
* use stderr for error messages, fixes #262
* fixed index out of bound, fixes #203
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants