eclipse-archived / ceylon.formatter

A formatter for the Ceylon programming language, written in Ceylon.
Apache License 2.0
14 stars 11 forks source link

Move tests to ceylon.formatter module #45

Closed lucaswerkmeister closed 10 years ago

lucaswerkmeister commented 10 years ago

The tests would be in a different source folder, but still in the same module, as suggested by Tom Bentley here. This allows you to use ceylon test ceylon.formatter instead of ceylon test test.ceylon.formatter, and more importantly, it allows us to make FormattingWriter and a bunch of other stuff unshared.

Depends on ceylon/ceylon-compiler#1572.

lucaswerkmeister commented 10 years ago

Damn, turns out we can’t unshare FormattingWriter because LineBreakStrategy depends on its internals.

lucaswerkmeister commented 10 years ago

Apart from some auxiliary functions in run.ceylon, the only thing we can unshare is FormattingVisitor. And somehow, it feels very wrong to me to have FormattingVisitor unshared but FormattingWriter shared.