On Mar 16, 2005, at 1:27 AM, Randy W. Sims wrote:
I'm not sure I'm comfortable with the new code for properties. It's
better than the original which I see from your comments that you were
not happy with either. But the new code is really just a more
complicated version of the original. I'm not sure I know a better
solution. I'm not even completely sure I understand all that's going
on there. I do know there is at least one problem which I posted about
around a week and a half ago. I haven't done anything more since then
because I've been nursing a cold. I guess it just doesn't feel right
having property data stored as class data. Since this code is not yet
in release, it seems good to explore options before getting locked in
to a particular implementation.
I took another brief look at this. The specific reason your test case
of 2/27/2005 is failing is that $self->{properties}{blib} is undefined
during new_from_context(). I *think* this is because all the
'properties' data is isolated per-class, rather than shared from
parent-class to child-class in the traditional perl hash-based OO way.
Or, maybe I don't understand it well enough.
If this is indeed the case though, I think we should be more
traditional.
-Ken
-------------------------------------------------------
This SF.net email is sponsored by: 2005 Windows Mobile Application Contest
Submit applications for Windows Mobile(tm)-based Pocket PCs or Smartphones
for the chance to win $25,000 and application distribution. Enter today at
http://ads.osdn.com/?ad_id=6882&alloc_id=15148&op=click
|
