[CDO] 0.8 Multiplr Inheritence Wrong featureID [message #117444] |
Wed, 09 April 2008 12:06  |
Eclipse User |
|
|
|
Hi Eike,
It seems that when we are using multiple inheritence we do not use the right
FeatureID.
In CDORevisionImpl we do the following.
int i = feature.getFeatureIndex();
Object old = values[i];
values[i] = value;
return old;
feature.getFeatureIndex(); will return the id of the current hierarchy. It
is wrong.
In CDOClassIMpl we seems to correctly calculate the ID in the variable
indice... but we do not use it in CDORevisionIMpl.
Do you want me to open a bugs ?
|
|
|
|
|
|
Re: [CDO] 0.8 Multiple Inheritence Wrong featureID [message #117805 is a reply to message #117781] |
Fri, 11 April 2008 15:35  |
Eclipse User |
|
|
|
you are welcome!!
Be sure to take version 3.. I've made few patches and the last one is the
most complete!!!
"Stefan Winkler" <stefan.winkler-et@fernuni-hagen.de> wrote in message
news:ftnac0$4th$1@build.eclipse.org...
> Cool, thank you for your patch.
> I hit that same bug only yesterday. After applying your latest patch, my
> code works fine again.
>
> Cheers,
> Stefan
>
> Simon McDuff schrieb:
>> OK I will come up with a Patch and testcases.
>>
>> https://bugs.eclipse.org/bugs/show_bug.cgi?id=226317
>>
>>
>> "Eike Stepper" <stepper@sympedia.de> wrote in message
>> news:ftirn8$shr$1@build.eclipse.org...
>>
>>> Hi Simon,
>>>
>>> If you think this is a bug I'm sure it is! Yes, please open a Bugzilla.
>>> Since I'm currently very (too) busy with my customer a patch would be
>>> most welcome ;-)
>>>
>>> Cheers
>>> /Eike
>>>
>>>
>>>
>>> Simon McDuff schrieb:
>>>
>>>> Hi Eike,
>>>>
>>>> It seems that when we are using multiple inheritence we do not use the
>>>> right FeatureID.
>>>>
>>>> In CDORevisionImpl we do the following.
>>>>
>>>> int i = feature.getFeatureIndex();
>>>>
>>>> Object old = values[i];
>>>>
>>>> values[i] = value;
>>>>
>>>> return old;
>>>>
>>>>
>>>>
>>>> feature.getFeatureIndex(); will return the id of the current hierarchy.
>>>> It is wrong.
>>>>
>>>>
>>>>
>>>> In CDOClassIMpl we seems to correctly calculate the ID in the variable
>>>> indice... but we do not use it in CDORevisionIMpl.
>>>>
>>>>
>>>>
>>>> Do you want me to open a bugs ?
>>>>
>>>>
>>
>>
|
|
|
Re: [CDO] 0.8 Multiplr Inheritence Wrong featureID [message #616147 is a reply to message #117444] |
Wed, 09 April 2008 12:43  |
Eclipse User |
|
|
|
Hi Simon,
If you think this is a bug I'm sure it is! Yes, please open a Bugzilla.
Since I'm currently very (too) busy with my customer a patch would be most welcome ;-)
Cheers
/Eike
Simon McDuff schrieb:
> Hi Eike,
>
> It seems that when we are using multiple inheritence we do not use the right
> FeatureID.
>
> In CDORevisionImpl we do the following.
>
> int i = feature.getFeatureIndex();
>
> Object old = values[i];
>
> values[i] = value;
>
> return old;
>
>
>
> feature.getFeatureIndex(); will return the id of the current hierarchy. It
> is wrong.
>
>
>
> In CDOClassIMpl we seems to correctly calculate the ID in the variable
> indice... but we do not use it in CDORevisionIMpl.
>
>
>
> Do you want me to open a bugs ?
>
>
|
|
|
Re: [CDO] 0.8 Multiplr Inheritence Wrong featureID [message #616153 is a reply to message #117471] |
Wed, 09 April 2008 13:13  |
Eclipse User |
|
|
|
OK I will come up with a Patch and testcases.
https://bugs.eclipse.org/bugs/show_bug.cgi?id=226317
"Eike Stepper" <stepper@sympedia.de> wrote in message
news:ftirn8$shr$1@build.eclipse.org...
> Hi Simon,
>
> If you think this is a bug I'm sure it is! Yes, please open a Bugzilla.
> Since I'm currently very (too) busy with my customer a patch would be most
> welcome ;-)
>
> Cheers
> /Eike
>
>
>
> Simon McDuff schrieb:
>> Hi Eike,
>>
>> It seems that when we are using multiple inheritence we do not use the
>> right FeatureID.
>>
>> In CDORevisionImpl we do the following.
>>
>> int i = feature.getFeatureIndex();
>>
>> Object old = values[i];
>>
>> values[i] = value;
>>
>> return old;
>>
>>
>>
>> feature.getFeatureIndex(); will return the id of the current hierarchy.
>> It is wrong.
>>
>>
>>
>> In CDOClassIMpl we seems to correctly calculate the ID in the variable
>> indice... but we do not use it in CDORevisionIMpl.
>>
>>
>>
>> Do you want me to open a bugs ?
>>
|
|
|
Re: [CDO] 0.8 Multiplr Inheritence Wrong featureID [message #616202 is a reply to message #117497] |
Fri, 11 April 2008 05:17  |
Eclipse User |
|
|
|
Cool, thank you for your patch.
I hit that same bug only yesterday. After applying your latest patch, my
code works fine again.
Cheers,
Stefan
Simon McDuff schrieb:
> OK I will come up with a Patch and testcases.
>
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=226317
>
>
> "Eike Stepper" <stepper@sympedia.de> wrote in message
> news:ftirn8$shr$1@build.eclipse.org...
>
>> Hi Simon,
>>
>> If you think this is a bug I'm sure it is! Yes, please open a Bugzilla.
>> Since I'm currently very (too) busy with my customer a patch would be most
>> welcome ;-)
>>
>> Cheers
>> /Eike
>>
>>
>>
>> Simon McDuff schrieb:
>>
>>> Hi Eike,
>>>
>>> It seems that when we are using multiple inheritence we do not use the
>>> right FeatureID.
>>>
>>> In CDORevisionImpl we do the following.
>>>
>>> int i = feature.getFeatureIndex();
>>>
>>> Object old = values[i];
>>>
>>> values[i] = value;
>>>
>>> return old;
>>>
>>>
>>>
>>> feature.getFeatureIndex(); will return the id of the current hierarchy.
>>> It is wrong.
>>>
>>>
>>>
>>> In CDOClassIMpl we seems to correctly calculate the ID in the variable
>>> indice... but we do not use it in CDORevisionIMpl.
>>>
>>>
>>>
>>> Do you want me to open a bugs ?
>>>
>>>
>
>
|
|
|
Re: [CDO] 0.8 Multiple Inheritence Wrong featureID [message #616206 is a reply to message #117781] |
Fri, 11 April 2008 15:35  |
Eclipse User |
|
|
|
you are welcome!!
Be sure to take version 3.. I've made few patches and the last one is the
most complete!!!
"Stefan Winkler" <stefan.winkler-et@fernuni-hagen.de> wrote in message
news:ftnac0$4th$1@build.eclipse.org...
> Cool, thank you for your patch.
> I hit that same bug only yesterday. After applying your latest patch, my
> code works fine again.
>
> Cheers,
> Stefan
>
> Simon McDuff schrieb:
>> OK I will come up with a Patch and testcases.
>>
>> https://bugs.eclipse.org/bugs/show_bug.cgi?id=226317
>>
>>
>> "Eike Stepper" <stepper@sympedia.de> wrote in message
>> news:ftirn8$shr$1@build.eclipse.org...
>>
>>> Hi Simon,
>>>
>>> If you think this is a bug I'm sure it is! Yes, please open a Bugzilla.
>>> Since I'm currently very (too) busy with my customer a patch would be
>>> most welcome ;-)
>>>
>>> Cheers
>>> /Eike
>>>
>>>
>>>
>>> Simon McDuff schrieb:
>>>
>>>> Hi Eike,
>>>>
>>>> It seems that when we are using multiple inheritence we do not use the
>>>> right FeatureID.
>>>>
>>>> In CDORevisionImpl we do the following.
>>>>
>>>> int i = feature.getFeatureIndex();
>>>>
>>>> Object old = values[i];
>>>>
>>>> values[i] = value;
>>>>
>>>> return old;
>>>>
>>>>
>>>>
>>>> feature.getFeatureIndex(); will return the id of the current hierarchy.
>>>> It is wrong.
>>>>
>>>>
>>>>
>>>> In CDOClassIMpl we seems to correctly calculate the ID in the variable
>>>> indice... but we do not use it in CDORevisionIMpl.
>>>>
>>>>
>>>>
>>>> Do you want me to open a bugs ?
>>>>
>>>>
>>
>>
|
|
|
Powered by
FUDForum. Page generated in 0.07047 seconds