fix: ensure sniper pretty-printer prints modifiers and type separated by a space

src/main/java/spoon/support/compiler/jdt/JDTTreeBuilder.java

@@ -1525,7 +1525,11 @@ public boolean visit(QualifiedTypeReference qualifiedTypeReference, BlockScope s
 			context.enter(helper.createCatchVariable(qualifiedTypeReference, scope), qualifiedTypeReference);
 			return true;
 		}
-		context.enter(factory.Code().createTypeAccessWithoutCloningReference(references.buildTypeReference(qualifiedTypeReference, scope)), qualifiedTypeReference);
+		CtTypeReference<?> typeReference = references.buildTypeReference(qualifiedTypeReference, scope);
+		if (typeReference != null) {
+			typeReference.setPosition(position.buildPositionCtElement(typeReference, qualifiedTypeReference));
+		}
+		context.enter(factory.Code().createTypeAccessWithoutCloningReference(typeReference), qualifiedTypeReference);
 		return true;
 	}
 

src/main/java/spoon/support/compiler/jdt/PositionBuilder.java

@@ -415,12 +415,6 @@ SourcePosition buildPositionCtElement(CtElement e, ASTNode node) {
 			//build position with appropriate context
 			return buildPositionCtElement(e, (Argument) pair.node);
 		} else if (node instanceof TypeReference) {
-			TypeReference typeReference = (TypeReference) node;
-			if (typeReference.resolvedType.getTypeAnnotations() != null) {
-				for (int a = 0; a < typeReference.resolvedType.getTypeAnnotations().length; a++) {
-					sourceStart = findPrevAnnotations(contents, 0, sourceStart);
-				}
-			}
 			sourceEnd = getSourceEndOfTypeReference(contents, (TypeReference) node, sourceEnd);
 		} else if (node instanceof AllocationExpression) {
 			AllocationExpression allocationExpression = (AllocationExpression) node;

src/test/java/spoon/test/issue3321/AnnotationPositionTest.java

@@ -16,6 +16,8 @@
 
 public class AnnotationPositionTest {
 
+	@Ignore("Unresolved Bug")
+	@GitHubIssue(issueNumber = 3358)
 	@Test
 	public void testUsageOfTypeAnnotationOnParameterInMethod() {
 		final Launcher launcher = new Launcher();

src/test/java/spoon/test/serializable/SourcePositionTest.java

@@ -28,6 +28,7 @@
 import org.junit.Test;
 
 import spoon.Launcher;
+import spoon.reflect.CtModel;
 import spoon.reflect.declaration.CtField;
 import spoon.reflect.declaration.CtType;
 import spoon.reflect.factory.Factory;
@@ -71,4 +72,18 @@ public void testSourcePosition() throws IOException {
 		CtField<?> elem2 = typeFromFile.getField("a");
 		assertTrue(elem1.getPosition().getFile().equals(elem2.getPosition().getFile()));
 	}
+
+	@Test
+	public void test_sourcePositionOfTypeInFieldExists() {
+		// contract: the type reference of type of the field should have a source position
+		final Launcher launcher = new Launcher();
+		launcher.addInputResource("src/test/resources/spoon/test/sourcePosition/FieldType.java");
+		launcher.addInputResource("src/test/resources/spoon/test/sourcePosition/SourcePartitionValidator.java");
+		CtModel model = launcher.buildModel();
+
+		CtField<?> field = (CtField<?>) model.getElements(
+				element -> element instanceof CtField &&
+						((CtField<?>) element).getSimpleName().equals("pleaseAttachSourcePositionToMyType")).stream().findFirst().get();
+		assertTrue("Source position unknown for type of field", field.getType().getPosition().isValidPosition());
+	}
 }

src/test/resources/spoon/test/sourcePosition/FieldType.java

@@ -0,0 +1,5 @@
+package sourcePosition;
+
+public class FieldType {
+    private SourcePartitionValidator.MatchingStrategy pleaseAttachSourcePositionToMyType;
+}

src/test/resources/spoon/test/sourcePosition/SourcePartitionValidator.java

@@ -0,0 +1,5 @@
+package sourcePosition;
+
+class SourcePartitionValidator {
+    public enum MatchingStrategy { }
+}