Am 14. März 2012 20:33 schrieb Benedikt Ritter <[email protected]>: > Am 14. März 2012 18:43 schrieb <[email protected]>: >> Author: sebb >> Date: Wed Mar 14 17:43:35 2012 >> New Revision: 1300661 >> >> URL: http://svn.apache.org/viewvc?rev=1300661&view=rev >> Log: >> CSV-60 CSVParser.iterator().remove() should throw throw new >> UnsupportedOperationException() >> > > is it reasonable to add a unit test, that makes sure an > UnsupportedOperationException is thrown? >
just saw the second commit on this one. Forget it. >> Modified: >> >> commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVParser.java >> >> Modified: >> commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVParser.java >> URL: >> http://svn.apache.org/viewvc/commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVParser.java?rev=1300661&r1=1300660&r2=1300661&view=diff >> ============================================================================== >> --- >> commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVParser.java >> (original) >> +++ >> commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/CSVParser.java >> Wed Mar 14 17:43:35 2012 >> @@ -212,7 +212,9 @@ public class CSVParser implements Iterab >> } >> } >> >> - public void remove() { } >> + public void remove() { >> + throw new UnsupportedOperationException(); >> + } >> }; >> } >> >> >> --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
