Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
[chemclipse-dev] Bug in org.eclipse.chemclipse.model.core.AbstractChromatogram.removeScans(int, int)

@Override
public void removeScans(int from, int to) {

int start;
int end;
if(from > to) {
start = to;
end = from;
} else {
start = from;
end = to;
}
for(int i = start; i <= end; i++) {
removeScan(start);     <====== Should be removeScan(i);
                                           // and in Java style, the "from" always included, the "to" always exclued.
}
/*
* Forces all listeners to be updated.
*/
// TODO Test updateChange
// fireUpdateChange(true);
}

Best regards,

Trig

Back to the top