search

lspitzner / brittany

haskell source code formatter
GNU Affero General Public License v3.0
690 stars 72 forks source link

Unboxed tuples #186

Closed sergv closed 6 years ago

sergv commented 6 years ago

I noticed that indenting snipptes like

case foo# of
  (# bar#, baz# #) -> ...

will produce output like

case foo# of
  (#bar#, baz##) -> ...

which is invalid. I decided to put spaces around delimiters in (#...#) unconditionally to avoid having to check whether names have hashes. This style also seems more prevalent from my non-representative sample.

I also enabled running literate tests in parallel on my machine and though it could be useful for others. Please let me know what you thing about using all available cores to run the tests.

ElvishJerricco commented 6 years ago

I think doing this unconditionally is pretty common style anyway, so I think this is a good change

lspitzner commented 6 years ago

Thanks! I agree that it makes sense to consistently use spaces for unboxed tuples.

I have pushed another commit with minor cleanups. Should be ready to merge.

Regarding the parallel testing, I am a bit confused about the failing test. The failure indicates a performance issue (and might be due to heavy load on travis) but then it is for a testsuite that you did not even touch. Plus it is only new-build that is failing, so I wonder if new-build is just somehow picking up the flag or running entire suits in parallel or something. Maybe I have to force serial test execution via -j1 for new-build on travis? If all that does not work I might prefer to revert that bit, but let's see first.