Remove "hacky" fix for CST Building while backtracking

[clementdessoude]

We can now remove this "hack" since it was fixed in the new releases of Chevrotain.

See:

prettier-java/packages/java-parser/src/parser.js

Lines 148 to 154 in 73a888d

	// hack to turn off CST building side effects during backtracking
	// TODO: should be patched in Chevrotain
	cstPostNonTerminal(ruleCstResult, ruleName) {
	if (this.isBackTracking() === false) {
	super.cstPostNonTerminal(ruleCstResult, ruleName);
	}
	}

and Chevrotain/chevrotain#789

I assign this to @Shaolans since he coded the prettier-ignore support which could be affected here. Please let me know if you don't have time to look at it, and I'll try to do it

[Shaolans]

@bd82 Does the node in ruleCstResult is built during backtracking ? Because if not it might fail due the fact that we need the location of the node for the comments.

[bd82]

I think the CSTNode is build but the position information is not added to it nor is it added to its parent CSTNode.

• Chevrotain/chevrotain@1dc658e#diff-2e10783ac7c5b55b9ab1cd01b7988691

You could disable comment tracking logic while backtracking as well...