[debbugs-tracker] bug#17657: closed (pretty-print #:width values above 49 don't work?)

[GNU bug Tracking System]

Your message dated Tue, 21 Jun 2016 16:47:20 +0200
with message-id <address@hidden>
and subject line Re: bug#17657: pretty-print #:width values above 49 don't work?
has caused the debbugs.gnu.org bug report #17657,
regarding pretty-print #:width values above 49 don't work?
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
17657: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=17657
GNU Bug Tracking System
Contact address@hidden with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> pretty-print #:width values above 49 don't work?
>
>
>
>
> Date: 
>
> Sat, 31 May 2014 11:24:12 -0700 (PDT)
>
>
>
>
> Hi,
>
> Neither example below should be line-wrapped, but the second one 
> is.  It looks like pretty-print #:width values above 49 don't
> work.  Is there a workaround?
>
>
> (pretty-print
>   '(234 . "901234567890123456789012345678901234567")
>   #:width 49) 
> =>
> (234 . "901234567890123456789012345678901234567")
>
> (pretty-print
>   '(234 . "9012345678901234567890123456789012345678")
>   #:width 50) 
> =>
> (234
>  .
>  "9012345678901234567890123456789012345678")
>
>
> $ ./config.guess
> x86_64-unknown-linux-gnu
>
> $ guile --version
> Guile 1.8.8
>
>
> Thanks!
> - Mark
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#17657: pretty-print #:width values above 49 don't work?
>
>
>
>
> Date: 
>
> Tue, 21 Jun 2016 16:47:20 +0200
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)
>
>
>
>
> Hi :)
>
> On Sun 01 Jun 2014 20:40, Mark H Weaver <address@hidden> writes:
>
> > Mark Polesky <address@hidden> writes:
> >> Neither example below should be line-wrapped, but the second one 
> >> is.
> >
> > Our pretty printer, inherited from SLIB and originally written by Marc
> > Feeley, uses a great many heuristics to make formatting decisions.
> > These heuristics are undocumented and subject to change.
> >
> > One of its heuristics is that individual expressions shall not be more
> > than 49 columns wide.  Notice that pretty-print.scm includes the
> > following internal definition:
> >
> >   (define max-expr-width 50)
> >
> > (although in fact the maximum width ends up being 49)
> >
> >> Is there a workaround?
> >
> > A couple of years ago I added an undocumented #:max-expr-width keyword
> > argument to pretty-print, in order to improve the formatting of
> > psyntax-pp.scm.  You could use that.
>
> Just pushed some documentation, so this bug is done.
>
> > It would be good to redesign and rewrite our pretty-printer from scratch
> > at some point.
>
> Yes!
>
> Andy
>
>
> --- End Message ---