Opened 6 years ago

Last modified 4 months ago

#165 reopened defect

pprint-logical-block fails for format with list directive

Reported by: mevenson Owned by: somebody
Priority: critical Milestone: 1.6.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]

Subtickets (add)

Attachments (1)

ticket-165.diff (3.7 KB) - added by mevenson 6 years ago.

Download all attachments as: .zip

Change History (18)

comment:1 Changed 6 years ago by mevenson

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 6 years ago by mevenson

comment:2 Changed 6 years ago by mevenson

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

comment:3 Changed 6 years ago by mevenson

  • Resolution set to fixed
  • Status changed from new to closed

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

comment:4 Changed 5 years ago by mevenson

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:5 Changed 5 years ago by mevenson

  • Keywords needs-triage added
  • Milestone changed from 0.27 to 1.1.0
  • Version changed from 0.27 to 1.1.0-dev

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

comment:6 Changed 5 years ago by mevenson

  • Keywords regression added
  • Priority changed from major to critical

comment:7 Changed 5 years ago by mevenson

  • Keywords changed from sbcl-buildhost,regression,needs-triage to sbcl-buildhost regression needs-triage

comment:8 Changed 5 years ago by mevenson

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 5 years ago by mevenson

  • 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 5 years ago by mevenson

  • Milestone changed from 1.1.0 to 1.1.1

comment:11 Changed 5 years ago by mevenson

  • Milestone changed from 1.1.1 to 1.2.0
  • Version changed from 1.1.0-dev to 1.2.0-dev

comment:12 Changed 4 years ago by https://www.google.com/accounts/o8/id?id=AItOawkYnNNEAO_K40Gp0xROhyjOPgjvIskQ48M

  • Milestone changed from 1.2.0 to 1.3.0

comment:13 Changed 4 years ago by mevenson

  • Milestone changed from 1.3.0 to 2.0

Ticket retargeted after milestone closed

comment:14 Changed 4 years ago by mevenson

  • Milestone changed from 2.0 to 2.0.0

Milestone renamed

comment:15 Changed 4 years ago by mevenson

  • Milestone changed from 2.0.0 to 1.4.0

comment:16 Changed 13 months ago by mevenson

  • Milestone changed from 1.4.0 to 1.5.0

Ticket retargeted after milestone closed

comment:17 Changed 4 months ago by mevenson

  • Milestone changed from 1.5.0 to 1.6.0

Ticket retargeted after milestone closed

Note: See TracTickets for help on using tickets.