9 Oct
2013
9 Oct
'13
12:46 a.m.
DiegoLont wrote
I converted my comments into code. Please take a look
at Magritte-Model,
and test if it works for you. I do not know if you have added a test for
this? All tests are green.
Thanks, Diego! I merged your changes. I just made a few minor edits -
removed the type check since blocks and symbols are polymorphic wrt #value:,
and changed the name to displayBlockOrSymbol (ugly I know, but displayBlock
seemed misleading).
I also wrote tests after you shamed me into it ;)
-----
Cheers,
Sean
--
View this message in context:
http://forum.world.st/Code-Review-Request-tp4711242p4713329.html
Sent from the Magritte, Pier and Related Tools mailing list archive at Nabble.com.