On May-30, Leopold Toetsch wrote:
> Steve Fink <[EMAIL PROTECTED]> wrote:
> 
> > Anyone mind if I commit this?
> 
> The patch is fine.
> 
> > ... One thing I'm not sure of, though -- I
> > try to behave myself and use Parrot_getenv rather than a plain
> > getenv(), but I'm not convinced the API is complete -- Parrot_getenv
> > saves back a boolean saying whether to free the returned string or
> > not, but what should I call to free it?
> 
> It's for Win32. config/gen/platform/win32/env.c uses
> C<mem_sys_allocate>, so it should be C<mem_sys_free>.

Thanks, patch applied. I went to add documentation on Parrot_getenv,
but found that it was already there in platform_interface.h. Doh!

Reply via email to