Re: NCI 't' parameter type removed; no replacement?

2011-05-13 Thread Patrick R. Michaud
On Fri, May 13, 2011 at 09:53:10PM -0400, Peter Lobsinger wrote: > On Fri, May 13, 2011 at 3:58 PM, Patrick R. Michaud > wrote: > > I also notice that according to api.yaml the UnManagedStruct type > > is deprecated and eligible for removal in 3.4 (i.e., "now"). > > Zavolaj uses that also, so in

Re: NCI 't' parameter type removed; no replacement?

2011-05-13 Thread Peter Lobsinger
On Fri, May 13, 2011 at 9:54 PM, Will Coleda wrote: > On Fri, May 13, 2011 at 9:44 PM, Peter Lobsinger wrote: >> On Fri, May 13, 2011 at 1:45 PM, Will Coleda wrote: >>> On Fri, May 13, 2011 at 10:21 AM, Peter Lobsinger >>> wrote: On Fri, May 13, 2011 at 7:24 AM, Jonathan Worthington

Re: NCI 't' parameter type removed; no replacement?

2011-05-13 Thread Will Coleda
On Fri, May 13, 2011 at 9:44 PM, Peter Lobsinger wrote: > On Fri, May 13, 2011 at 1:45 PM, Will Coleda wrote: >> On Fri, May 13, 2011 at 10:21 AM, Peter Lobsinger wrote: >>> On Fri, May 13, 2011 at 7:24 AM, Jonathan Worthington >>> wrote: Hi, The deprecated 't' parameter in NCI b

Re: NCI 't' parameter type removed; no replacement?

2011-05-13 Thread Peter Lobsinger
On Fri, May 13, 2011 at 3:58 PM, Patrick R. Michaud wrote: > I also notice that according to api.yaml the UnManagedStruct type > is deprecated and eligible for removal in 3.4 (i.e., "now"). > Zavolaj uses that also, so in order to avoid another "surprise" > we need details on the upgrade path for

Re: NCI 't' parameter type removed; no replacement?

2011-05-13 Thread Peter Lobsinger
On Fri, May 13, 2011 at 1:45 PM, Will Coleda wrote: > On Fri, May 13, 2011 at 10:21 AM, Peter Lobsinger wrote: >> On Fri, May 13, 2011 at 7:24 AM, Jonathan Worthington >> wrote: >>> Hi, >>> >>> The deprecated 't' parameter in NCI broke Zavolaj (the Perl 6 native call >>> interface). This in turn

Re: NCI 't' parameter type removed; no replacement?

2011-05-13 Thread Patrick R. Michaud
On Fri, May 13, 2011 at 01:45:51PM -0400, Will Coleda wrote: > [...] > This seems to fall under this clause, and I heartily agree. > > Let's bring this back, fix Zavolaj, document the upgrade path, > IMPLEMENT IT, and then we can worry about deleting the old stuff. +1. I also notice that accordi

Re: Re: NCI 't' parameter type removed; no replacement?

2011-05-13 Thread Jim Keenan
May 13, 2011 01:53:53 PM, w...@coleda.com wrote: > This seems to fall under this clause, and I heartily agree. > Let's bring this back, fix Zavolaj, document the upgrade path, > IMPLEMENT IT, and then we can worry about deleting the old stuff. I agree as well.

Re: NCI 't' parameter type removed; no replacement?

2011-05-13 Thread Patrick R. Michaud
On Fri, May 13, 2011 at 10:21:36AM -0400, Peter Lobsinger wrote: > On Fri, May 13, 2011 at 7:24 AM, Jonathan Worthington > To update your code, I would recommend making use of > Parrot_str_to_cstring/Parrot_str_free_cstring. These can be accessed > from HLL code using the NCI interface. > [...] > S

Re: NCI 't' parameter type removed; no replacement?

2011-05-13 Thread Will Coleda
On Fri, May 13, 2011 at 10:21 AM, Peter Lobsinger wrote: > On Fri, May 13, 2011 at 7:24 AM, Jonathan Worthington > wrote: >> Hi, >> >> The deprecated 't' parameter in NCI broke Zavolaj (the Perl 6 native call >> interface). This in turn broke various modules, including MiniDBI. Looking >> at: >>

Re: NCI 't' parameter type removed; no replacement?

2011-05-13 Thread Peter Lobsinger
On Fri, May 13, 2011 at 10:29 AM, Carl Mäsak wrote: > Jonathan (>>), Peter (>): >>> It seems that the signatures for e.g. mysql database connection were >>> removed, not updated! The deprecation notice doesn't provide anything that >>> seems to help with our use case. Replacing t with S seems to l

Re: NCI 't' parameter type removed; no replacement?

2011-05-13 Thread Jonathan Worthington
On 13/05/2011 16:29, Carl Mäsak wrote: Jonathan (>>), Peter (>): It seems that the signatures for e.g. mysql database connection were removed, not updated! The deprecation notice doesn't provide anything that seems to help with our use case. Replacing t with S seems to lead to segfaults; a f

Re: NCI 't' parameter type removed; no replacement?

2011-05-13 Thread Carl Mäsak
Jonathan (>>), Peter (>): >> It seems that the signatures for e.g. mysql database connection were >> removed, not updated! The deprecation notice doesn't provide anything that >> seems to help with our use case. Replacing t with S seems to lead to >> segfaults; a further look shows that seems not t

Re: NCI 't' parameter type removed; no replacement?

2011-05-13 Thread Peter Lobsinger
On Fri, May 13, 2011 at 7:24 AM, Jonathan Worthington wrote: > Hi, > > The deprecated 't' parameter in NCI broke Zavolaj (the Perl 6 native call > interface). This in turn broke various modules, including MiniDBI. Looking > at: > > https://github.com/parrot/parrot/commit/75e2b4c94d602d3d7e8af4b679

NCI 't' parameter type removed; no replacement?

2011-05-13 Thread Jonathan Worthington
Hi, The deprecated 't' parameter in NCI broke Zavolaj (the Perl 6 native call interface). This in turn broke various modules, including MiniDBI. Looking at: https://github.com/parrot/parrot/commit/75e2b4c94d602d3d7e8af4b679c325c8ec8d3fd9#src/nci/extra_thunks.nci It seems that the signatures