On 5/30/2014 8:06 PM, Karsten Bräckelmann wrote:
Cc'ing David, since I don't know if he is subscribed to dev@.
I am in favor of changing that code as David proposed: Shrink lots of
consecutive newlines to an upper limit, removing only excess lines.
20 lines seems reasonable to me. 10 is definitely too low, whereas 100
feels slightly excessive.
The proposed code changes are sufficiently simple to not require a CLA.
Committed to rev1600261. David, I added you to the credits. Good catch
on that bug!
+ - David B Funk <dbfunk(at)engineering.uiowa.edu>: Fix for newline collapse
+ causing excessive whitespace rules to not match
+