Chris Cunningham wrote
> Any chance I could get an account for that? I'm interested in the answer,
> but do not have a login to fogbugz.
I was going to say you can read it on Penelope [1] in the mean time, but
10220 does not appear there... Anyway, near the top of the page at [1]
you'll see a link to register for fogbugz (I'd love to give you a direct
link, but there's no permalink... hint, hint ;) [2]). Here's the current
discussion while you're waiting...
Edited by Clement Bera 06/06/2013 (Today) 12:29 PM
Image Version changed from (No Value) to 'Pharo3.0'
This is in
DynamicMessageImplementor>>#writeSourceCode
self writeMethodName.
self writeShouldBeImplemented.
self writeGetterSourceCodeIfNecessary.
self writeSetterSourceCodeIfNecessary
There should be a condition for writeShouldBeImplemented. Something like :
self isMessageAGetter | self isMessageASetter ifFalse: [ self
writeShouldBeImplemented. ]
It seems that this fix it here has no side effect.
However I don't like this condition, smells like dirty code. Perhaps we
should add some strategies for getter and setter ?
Edited by Sean DeNigris 06/06/2013 (Today) 12:49 PM
"I don't like this condition, smells like dirty code"
Agreed. We're building a nice little hidden switch statement here...
[1] http://bugs.pharo.org/
[2]
https://pharo.fogbugz.com/f/cases/10262/Permalink-to-Penelope-registration-page
-----
Cheers,
Sean
--
View this message in context:
http://forum.world.st/Debugger-why-generated-accessors-shouldBeImplemented-tp4691992p4692000.html
Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com.
