yeah, if you don't care to track the number of conflicts you should either break here, or as @timwr said, just return true.  I think idiomatic ruby would say use the break because it's harder to debug a method that has returns all over the place.  I personally think that's lame however, but that's what the smarter-than-me people say...