I think executor.setExitValues(null) belongs in this method, because it directly affects the behaviour of executor.execute() below, and thus the exitValues which must be handled in the switch statement.

You might find it easier to read the code in an IDE (or click "View File @ c456de5"), since the comments look a lot better in IDEs than in monotone diffs.  And personally, I disagree with the word "bloated" for these comments. ;-)