Hi Martin,
Indeed, the init() call is missing.
I'm impressed! :)
On Feb 12, 2010, at 8:37 PM, Martin Scholl wrote:
> Martin Scholl has proposed merging lp:~ms-fk/pbxt/trunk into lp:pbxt. > > Requested reviews: > PBXT Core (pbxt-core) > > > Hi all, > > > please verify this is actually needed; having read the surrounding > code, it seems so. > > > HiH, > Martin > -- > https://code.launchpad.net/~ms-fk/pbxt/trunk/+merge/19196 > Your team PBXT Core is subscribed to branch lp:pbxt. > === modified file 'src/myxt_xt.cc' > --- src/myxt_xt.cc 2010-01-22 13:24:35 +0000 > +++ src/myxt_xt.cc 2010-02-12 19:37:18 +0000 > @@ -2859,6 +2859,7 @@ > for (key_part = key->key_part; key_part != key_part_end; key_part+ > +) { > if (!(cref = new XTDDColumnRef())) > xt_throw_errno(XT_CONTEXT, XT_ENOMEM); > + cref->init(self); > ind->co_cols.append(self, cref); > cref->cr_col_name = xt_dup_string(self, (char *) key_part->field- > >field_name); > } >
-- Paul McCullagh PrimeBase Technologies www.primebase.org www.blobstreaming.org pbxt.blogspot.com
« Back to merge proposal
Hi Martin,
Indeed, the init() call is missing.
I'm impressed! :)
On Feb 12, 2010, at 8:37 PM, Martin Scholl wrote:
> Martin Scholl has proposed merging lp:~ms-fk/pbxt/trunk into lp:pbxt. /code.launchpad .net/~ms- fk/pbxt/ trunk/+ merge/19196 errno(XT_ CONTEXT, XT_ENOMEM); cols.append( self, cref);
>
> Requested reviews:
> PBXT Core (pbxt-core)
>
>
> Hi all,
>
>
> please verify this is actually needed; having read the surrounding
> code, it seems so.
>
>
> HiH,
> Martin
> --
> https:/
> Your team PBXT Core is subscribed to branch lp:pbxt.
> === modified file 'src/myxt_xt.cc'
> --- src/myxt_xt.cc 2010-01-22 13:24:35 +0000
> +++ src/myxt_xt.cc 2010-02-12 19:37:18 +0000
> @@ -2859,6 +2859,7 @@
> for (key_part = key->key_part; key_part != key_part_end; key_part+
> +) {
> if (!(cref = new XTDDColumnRef()))
> xt_throw_
> + cref->init(self);
> ind->co_
> cref->cr_col_name = xt_dup_string(self, (char *) key_part->field-
> >field_name);
> }
>
-- ng.org
Paul McCullagh
PrimeBase Technologies
www.primebase.org
www.blobstreami
pbxt.blogspot.com