Opened 13 years ago

Closed 4 years ago

#165 closed defect (fixed)

pprint-logical-block fails for format with list directive

Reported by: Mark Evenson Owned by: somebody
Priority: critical Milestone: 1.8.0
Component: other Version: 1.2.0-dev
Keywords: streams sbcl-buildhost regression needs-triage Cc:
Parent Tickets:

Description

Derived from the problems getting SBCL to compile, the following test fails with complaints in the XP package:

(defparameter *stream* *error-output*)

(defun pprint-test ()
  (pprint-logical-block (*stream* nil :prefix "---") 
    (format *stream* "~(~A~)" '(1 2 3 4))))

with this error

The value #S(XP::XP-STRUCTURE :BASE-STREAM #S(SLIME-OUTPUT-STREAM) :LINE-LENGTH 70 :LINE-LIMIT NIL :LINE-NO 1 :DEPTH-IN-BLOCKS 2 :BLOCK-STACK #(0 0 3 NIL NIL NIL NIL NIL NIL NIL ...) :BLOCK-STACK-PTR 2 :BUFFER … is not of type STREAM.
[Condition of type TYPE-ERROR]

Attachments (1)

ticket-165.diff (3.7 KB) - added by Mark Evenson 13 years ago.

Download all attachments as: .zip

Change History (25)

comment:1 Changed 13 years ago by Mark Evenson

ehu via #abcl

12:54 <ehu> easyE: the issue is on line 1076 of format.lisp                     
12:55 <ehu> but still, the real issue is that our xp printer isn't              
            a stream 

Changed 13 years ago by Mark Evenson

Attachment: ticket-165.diff added

comment:2 Changed 13 years ago by Mark Evenson

Behavior remove with attached patch. See notes in patch for why this isn't completely satisfied.

comment:3 Changed 13 years ago by Mark Evenson

Resolution: fixed
Status: newclosed

Fixed in r13538, whose commit message incorrectly states that it is only partially correct patch.

comment:4 Changed 12 years ago by Mark Evenson

Resolution: fixed
Status: closedreopened

comment:5 Changed 12 years ago by Mark Evenson

Keywords: needs-triage added
Milestone: 0.271.1.0
Version: 0.271.1.0-dev

Bug has resurfaced as of /trunk/abcl@13932.

comment:6 Changed 12 years ago by Mark Evenson

Keywords: regression added
Priority: majorcritical

comment:7 Changed 12 years ago by Mark Evenson

Keywords: sbcl-buildhost,regression,needs-triagesbcl-buildhost regression needs-triage

comment:8 Changed 12 years ago by Mark Evenson

The current problem is now that the prefix appears at the end of the stream.

The test has never worked, so the original comment on the commit about being a partial solution was somewhat correct.

comment:9 Changed 12 years ago by Mark Evenson

Keywords: streams added

Need to fix streams for good, now that we have a proper MOP to re-initialize the class structure properly.

c.f. http://trac.common-lisp.net/armedbear/wiki/implementation/components/streams

comment:10 Changed 12 years ago by Mark Evenson

Milestone: 1.1.01.1.1

comment:11 Changed 12 years ago by Mark Evenson

Milestone: 1.1.11.2.0
Version: 1.1.0-dev1.2.0-dev

comment:12 Changed 11 years ago by Evenson Not Org

Milestone: 1.2.01.3.0

comment:13 Changed 11 years ago by Mark Evenson

Milestone: 1.3.02.0

Ticket retargeted after milestone closed

comment:14 Changed 11 years ago by Mark Evenson

Milestone: 2.02.0.0

Milestone renamed

comment:15 Changed 11 years ago by Mark Evenson

Milestone: 2.0.01.4.0

comment:16 Changed 8 years ago by Mark Evenson

Milestone: 1.4.01.5.0

Ticket retargeted after milestone closed

comment:17 Changed 7 years ago by Mark Evenson

Milestone: 1.5.01.6.0

Ticket retargeted after milestone closed

comment:18 Changed 5 years ago by Mark Evenson

Milestone: 1.6.01.6.1

Ticket retargeted after milestone closed

comment:19 Changed 4 years ago by Mark Evenson

Milestone: 1.6.11.6.2

Ticket retargeted after milestone closed

comment:20 Changed 4 years ago by Mark Evenson

Milestone: 1.6.21.7.0

comment:21 Changed 4 years ago by Mark Evenson

Milestone: 1.7.01.7.1

Ticket retargeted after milestone closed

comment:22 Changed 4 years ago by Mark Evenson

Milestone: 1.7.11.7.2

Ticket retargeted after milestone closed

comment:23 Changed 4 years ago by Mark Evenson

Milestone: 1.7.21.8.0

Milestone renamed

comment:24 Changed 4 years ago by Mark Evenson

Resolution: fixed
Status: reopenedclosed
Note: See TracTickets for help on using tickets.