What do you mean non empty controllers?

```ruby
# routes.rb
get 'foobar' => 'foo#bar'

# foo_controller.rb
class FooController < ApplicationController
  def bar; end # this line is unnecessary
end
```

That's all I meant.

`get` is better than `match` because it's more specific ( match only GET request, not POST, PUT, PATCH, DELETE or anything else). The idea is that we should be creating exactly the routes we need and nothing more. Run `rake routes` sometime to see everything our app will try to route. This PR cleans that up **a lot**.

Unused routes are bad.