zzh
/
gatk-3.8
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Optimization to method for getting values in ArgumentMatch 

* Very trivial, but I happened to see this code and it drove me nuts so I felt compelled to refactor it.
  * Instead of iterating over keys in map to get the values, just iterate over the values...
This commit is contained in:
Eric Banks 2013-04-03 17:37:19 -05:00
parent 03811b4274
commit 14bbba0980
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
public/java/src/org/broadinstitute/sting/commandline/ArgumentMatch.java
View File

@ -276,10 +276,10 @@ public class ArgumentMatch implements Iterable<ArgumentMatch> {
* @return A collection of the string representation of these value. * @return A collection of the string representation of these value.
*/ */
public List<ArgumentMatchValue> values() { public List<ArgumentMatchValue> values() {
List<ArgumentMatchValue> values = new ArrayList<ArgumentMatchValue>(); final List<ArgumentMatchValue> values = new ArrayList<ArgumentMatchValue>();
for( ArgumentMatchSite site: sites.keySet() ) { for ( final List<ArgumentMatchValue> siteValue : sites.values() ) {
if( sites.get(site) != null ) if ( siteValue != null )
values.addAll(sites.get(site)); values.addAll(siteValue);
} }
return values; return values;
} }