CheckSignatureTask.process() does not sort file names

[Marcono1234]

CheckSignatureTask.process() does not sort the path elements, it just processes them in the order they are returned by Path.list():

animal-sniffer/animal-sniffer-ant-tasks/src/main/java/org/codehaus/mojo/animal_sniffer/ant/CheckSignatureTask.java

Lines 215 to 219 in 364fc20

	final String[] files = path.list();
	for ( int j = 0; j < files.length; j++ )
	{
	signatureChecker.process( new File( files[j] ) );
	}

This causes issues when the list first contains the file name of a nested class and afterwards the file name of the enclosing class. In that case forbidden API usage could be detected for the nested class despite the enclosing class being annotated. Other methods first sort the file names to make sure enclosing classes are processed first (e.g. here).

Is that intended? I am not quite sure how org.apache.tools.ant.types.Path is supposed to be used. Is it legit to directly store class file names in it, or is it only intended to be a collection of directory paths?

[Marcono1234]

@slachiewicz, it looks like this specific code was not changed. So is this not considered an issue, and path should only contain the directories to be checked, but not individual files? (as mentioned in xvik/gradle-animalsniffer-plugin#25)

Or did you close this issue by accident?

[slachiewicz]

Ok, by mistake only - cleanup. Pr welcome