On Wednesday 28 May 2008 11:58:02 Patrick R. Michaud via RT wrote:
> On Mon Oct 22 09:37:22 2007, pcoch wrote:
> > In src/pmc/default.pmc:get_attr_str() there is the todo item:
> >
> > if (PMC_IS_NULL(p)) {
> > /* TODO AttributeError */
> > }
> >
> > This should be as simple as adding a real_exception() here.
>
> I vote to reject this TODO item. Returning a null PMC is pretty much
> the standard Parrot approach for indicating a non-existent value (i.e.,
> that's what we do for arrays, hashes, symbols in a namespace, etc.).
Works for me. I tidied that function somewhat in r27890, and all tests pass.
-- c
