fix: Remove lineSeparatorPositions check

[Egor18]

Hello.
I was trying to migrate from spoon 7.0 to 7.2, and encountered some mode build failures due to the recent changes. (We have a large test base with ~80 open-source projects).

Spoon 7.2. fails to build a model with spoon.SpoonException: Unexpected lineSeparatorPositions for the file with commented out class (the code is from jmonkeyengine project):

package com.jme3.bounding;

//public class OrientedBoundingBox {
// ...
//}

However, that file compiles successfully, and spoon 7.0 was able to handle it.
I think this issue is related to that commit: f68c62c

So I'd like commented out classes not to fail an entire model build.

[Egor18]

It seems like SpoonArchitectureEnforcerTest.testGoodTestClassNames fails to build a model for a commented out class:

java.lang.NullPointerException
	at spoon.support.compiler.jdt.JDTTreeBuilderQuery.searchTypeBinding(JDTTreeBuilderQuery.java:85)
	at spoon.support.compiler.jdt.ReferenceBuilder.getQualifiedTypeReference(ReferenceBuilder.java:283)
	at spoon.support.compiler.jdt.JDTTreeBuilderHelper.createTypeAccess(JDTTreeBuilderHelper.java:438)
	at spoon.support.compiler.jdt.JDTTreeBuilderHelper.createTargetFieldAccess(JDTTreeBuilderHelper.java:538)
	at spoon.support.compiler.jdt.JDTTreeBuilderHelper.createVariableAccess(JDTTreeBuilderHelper.java:271)

Other tests are ok. I can remove the test case, but leave the change. WDYT?

[pvojtechovsky]

I am having a look at this.

[monperrus]

Thanks a lot @Egor18 for the failing test case. Closed by #2824.