[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
|