[ENH] Hook up non-transforming pretty-printing

Doug Way dway at riskmetrics.com
Fri Nov 30 06:31:56 UTC 2001


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 at riskmetrics.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: NoInlineParserAdds-dew.cs.gz
Type: application/octet-stream
Size: 832 bytes
Desc: Unknown Document
Url : http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20011130/f3a400bf/NoInlineParserAdds-dew.cs.obj


More information about the Squeak-dev mailing list