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 > > > >
signature.asc
Description: PGP signature