xtext clean output - not removing all output folders, if previous does not exists

Hi,
I would like to report a possible bug. Please take a look at line 490 in org.eclipse.xtext.builder.BuilderParticipant -> cleanOutput(IBuildContext, OutputConfiguration, EclipseResourceFileSystemAccess2, IProgressMonitor)

https://github.com/eclipse/xtext/blob/cab7c302fcdc9b8975d58b12061da06c12ec369c/org.eclipse.xtext.builder/src/org/eclipse/xtext/builder/BuilderParticipant.java#L490

This method removes output folders based on configuration: the general logic is:
for (IContainer container : getOutputs(project, config)) {
delete the output
}

The problem is with the following lines:
for (IContainer container : getOutputs(project, config)) {
if (!container.exists()) {
return;
}

It stops deleting on the first non-existing resource to be deleted. So, if getOuputs returns two folders, and both exists - both will be deleted. If the first one does not exists, but the second does - it will not be deleted.

I propose to change "return" into "continue" - this should fix the issue.
Kind regards
Marcin

Start at line 490 in org.eclipse.xtext.builder/src/org/eclipse/xtext/builder/BuilderParticipant.java, within cleanOutput(IBuildContext, OutputConfiguration, EclipseResourceFileSystemAccess2, IProgressMonitor), and inspect how the getOutputs(project, config) loop handles missing containers. Done means cleanup continues past a missing first output and removes later existing output folders.