#9 Newlines Broken in CCL

Open
opened 8 months ago by jgodbout · 8 comments

In Clozure, and perhaps other lisps (I couldn't try), the report format differs from that in the tutorial. Newlines are missing, e.g.:

(cls-tests:run)

PROGRESS: TESTS: (Test Suite)

COMPOUND: (Test Suite)

    ARITHMETIC: (Test Suite)
        VECTOR-ADDITION: F.

FAILURE DETAILS:================TESTS -> COMPOUND -> ARITHMETIC: (Test Suite) VECTOR-ADDITION: Expression: (= 1 -1) Expected: T Returned: NIL SUMMARY:========Test functions:Executed: 1Skipped: 0Tested 2 assertions.Passed: 1/2 ( 50.0%)Failed: 1/2 ( 50.0%)

Bug originally stated by snunez1 at: https://github.com/tgutu/clunit/issues/11

In Clozure, and perhaps other lisps (I couldn't try), the report format differs from that in the tutorial. Newlines are missing, e.g.: (cls-tests:run) PROGRESS: TESTS: (Test Suite) COMPOUND: (Test Suite) ARITHMETIC: (Test Suite) VECTOR-ADDITION: F. FAILURE DETAILS:================TESTS -> COMPOUND -> ARITHMETIC: (Test Suite) VECTOR-ADDITION: Expression: (= 1 -1) Expected: T Returned: NIL SUMMARY:========Test functions:Executed: 1Skipped: 0Tested 2 assertions.Passed: 1/2 ( 50.0%)Failed: 1/2 ( 50.0%) Bug originally stated by snunez1 at: https://github.com/tgutu/clunit/issues/11
cage commented 8 months ago
Owner

Uhm...this is strange, i fear there will be needs for a bunch of #+ccl to fix this bug, OK i have to install CCL and i am going to start investigating this.

Bye! C.

Uhm...this is strange, i fear there will be needs for a bunch of `#+ccl` to fix this bug, OK i have to install CCL and i am going to start investigating this. Bye! C.
jgodbout commented 8 months ago
Poster

This affects abcl as well

This affects abcl as well
cage commented 8 months ago
Owner

Hi jgodbout!

Now i wonder if setting the special variable *print-pretty* to t would be enough (perhaps for CCL and ABCL the default value is nil).

Bye!

C.

Hi jgodbout! Now i wonder if setting the special variable `*print-pretty*` to `t` would be enough (perhaps for `CCL` and `ABCL` the default value is `nil`). Bye! C.
cage commented 8 months ago
Owner

Hi jgodbout!

Fortunately at least with CCL (v1.12) setting *print-pretty* to t seems to works! :)

Please could you just try the same on ABCL, i feel the same setting works on this implementation.

Bye!

C.

Hi jgodbout! Fortunately at least with CCL (v1.12) setting `*print-pretty*` to `t` seems to works! :) Please could you just try the same on ABCL, i feel the same setting works on this implementation. Bye! C.
jgodbout commented 8 months ago
Poster

Doesn't seem to work with abcl...

Doesn't seem to work with abcl...
cage commented 8 months ago
Owner

Hi jgodbout!

Oh, that's bad :(

So seems that I have to setup ABCL an check what's going wrong.

Thank you for trying by the way!

Bye!

C.

Hi jgodbout! Oh, that's bad :( So seems that I have to setup ABCL an check what's going wrong. Thank you for trying by the way! Bye! C.
cage commented 8 months ago
Owner

Hi jgodbout!

It is very far from perfect but i pushed a workaround that, in my opinion, could make the output this library (when using ABCL) less messy.

To works you have to bind *print-pretty* to t.

Feel free to comment, as usual! :)

Bye!

C.

Hi jgodbout! It is very far from perfect but i pushed a workaround that, in my opinion, could make the output this library (when using ABCL) less messy. To works you have to bind `*print-pretty*` to `t`. Feel free to comment, as usual! :) Bye! C.
cage commented 8 months ago
Owner

Hi jgodbout!

Can you confirm that this patch fix the issues with ABCL?

Bye!

C.

Hi jgodbout! Can you confirm that this patch fix the issues with ABCL? Bye! C.
Sign in to join this conversation.
No Milestone
No assignee
2 Participants
Loading...
Cancel
Save
There is no content yet.