Description
Going through PMD output, and this looks a little sketchy in SourceSwitchingIterator.java:
private boolean switchSource() throws IOException { while (!source.isCurrent()) { source = source.getNewDataSource(); iter = source.iterator(); if (iflag != null) ((InterruptibleIterator) iter).setInterruptFlag(iflag); return true; } return false; }
I'm wondering if that "while" should be "if"