Conversation
|
thank you, strange for me why tests are failing... |
|
This broke the unexpand tests |
|
This failure is somewhat hidden because test/common/util.rs fails to compile with latest nightly. Can be fixed with: |
|
The problem is that the new version treats '\t' '\n' as 0 width characters. |
|
Ah ha, this must have been a subtle difference between and I'm not sure what the right answer is, exactly, should we just test for Also, how did you get that expanded test output, so I can see for myself? |
|
On Mon, Sep 28, 2015 at 07:35:17AM -0700, Steve Klabnik wrote:
Hmm, I only tested \n \t probably other control characters are also
|
On a second look I think these 2 are fundamentally different. |
built on top of #690, this removes some feature gates, in an effort to get closer to stable.
one or two more of these were trickier, so it's not fully on stable yet.