search.rb cleanups

- whitespace
- variable names: in particular, "casks" is a poor name for a
  variable that does not hold instances of Cask
Merge pull request #2495 from rolandwalker/search_cleanup

search.rb cleanups