Catch-22 with class containing nothing but protected or private methods

[lee-dohm]

I have a subclass in my current Rails project that looks approximately like so:

# Test class.
class Foo
  private

  def bar
    puts 'bar'
  end
end

For which I now (on master: 7b8e9ab ) get the warning:

C: Keep a blank line before and after private.

But if I do as it says and add the blank line before private, I get the following message:

C: Extra blank line detected at body beginning.

I think the EmptyLinesAroundAccessModifier cop should be relaxed by not requiring a blank line before the access modifier if it is the first semantic element in the class body.

[bbatsov]

Yeah, that's a bug in the check for empty lines around access modifiers. I knew this would be problematic, but it took some time before some complained about it. :-) I'll have that fixed.