Description Mikaël Barbero 2011-05-23 06:17:29 EDT

I got a corner case when trying to compare identical resources, ModelContentMergeDiffTab#mapDifference() is throwing me an ArrayIndexOutOfBoundException.

The else statement should test the items array of the tree for emptyness before accessing it. The comment says to do something else, but I don't see what. Maybe the real fix is not the test for emptyness, but to remove the TODO ;)

} else {
	// TODO for now, we're using the first item's data, we should
	// look for the matchedElement
	TreeItem[] items = getTree().getItems();
	if (items.length > 0) {
		dataToDiff.put(items[0].getData(), diff);
	}
}