[prev in list] [next in list] [prev in thread] [next in thread] 

List:       squeak-dev
Subject:    [ENH] Hook up non-transforming pretty-printing
From:       Doug Way <dway () riskmetrics ! com>
Date:       2001-11-30 6:31:56
[Download RAW message or body]


Here's a changeset which hooks up Henrik's non-transforming parser changes so that \
the pretty printer and tile scriptor use them.

Before filing this in, you'll need to file in Henrik's changesets from Monday.  See:

http://groups.yahoo.com/group/squeak/message/36671

This new pretty-printing seems to clean up most of the transformation problems.  I \
tested pretty-printing in the browsers, debugger, with diffs, and with tiles.

(To turn on pretty printing, use the shift-menu in the browser code pane, or set the \
preference, or the 'what to show' item, or... ;-) )

Here's an example method which shows the differences in how the pretty-printer worked \
before and after:

myMethod
	"one"
	| hey |
	"two"
	hey _ hey
			ifTrue: [nil]
			ifFalse: ['second'].
	"three"
	^ hey
		ifFalse: ['first']
		ifTrue: ['second'].

Previously, the first ifTrue: clause would dissappear completely, and the second \
ifFalse:ifTrue: would get switched.  Now this doesn't happen, although I notice that \
the 'nil' in the first block dissappears... not sure if this is a buglet.

Also, another general oddity is that the "two" comment moves before the temp var \
declarations (which also happened before).

- Doug Way
  dway@riskmetrics.com


["NoInlineParserAdds-dew.cs.gz" (application/octet-stream)]

[prev in list] [next in list] [prev in thread] [next in thread]