On Sun, 17 Jun 2007 01:41:30 -0500
Andy Lester <[EMAIL PROTECTED]> wrote:

> I cringe at functions where the behavior is dependent on parms
> passed in.  In the case of Parrot_get_runtime_prefix, if you call it
> as Parrot_get_runtime_prefix(&str) then str is populated, or if you
> call Parrot_get_runtime_prefix(NULL) then you get a malloced string.
> 
> It should be split into Parrot_get_runtime_prefix_string and  
> Parrot_get_runtime_prefix_cstring.
> 
> Anything I'm missing?  Dissent?

I have a patch already for it. I pulled it from RT because there was
a mistake in it, but I will re-send a new one today.

> xoxo,
> Andy
> 
> --
> Andy Lester => [EMAIL PROTECTED] => www.petdance.com => AIM:petdance
> 
> 
> 
> 

Attachment: signature.asc
Description: PGP signature

Reply via email to