remove redundant 'val' keyword
use .count instead of .filter.length

intellij suggests this; seems cleaner/simpler
add EoF newlines
break long-line of inheritance declarations
use nonEmpty instead of !.empty
simplify match statement, avoid return statement

IIRC, return statements are implemented using exceptions, and are
relatively unperformant as a result; also stylistically at odds with
functional code flow
group loci statements to match other callers

in some world we may want to reuse this boilerplate across VCs
fix variable shadowing

intellij doesn't like shadowed var names, seems reasonable
remove redundant/shadowed import
use .sort.reverse instead of .sort(-1 * _)
remove unnecessary _'s
add names to some cryptic arguments
collapse some .view.count()'s to just .count()
use ".exists()" instead of ".count() > 0"
Merge pull request #143 from hammerlab/style

some random style nits