Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cdt-dev] API tooling errors

We could probably revert the change to those classes to be totally safe, although it would be less elegant.
Let's wait for Mikhail to give his recommendation.
________________________________________
From: cdt-dev-bounces@xxxxxxxxxxx [cdt-dev-bounces@xxxxxxxxxxx] On Behalf Of Doug Schaefer [cdtdoug@xxxxxxxxx]
Sent: March 1, 2012 9:17 AM
To: CDT General developers list.
Subject: Re: [cdt-dev] API tooling errors

I do know of vendors subclassing the GDBControl classes. We need to be
absolutely sure it is compatible.

On Thu, Mar 1, 2012 at 9:14 AM, Marc Khouzam <marc.khouzam@xxxxxxxxxxxx> wrote:
>
> I'm starting to worry about the API tool.  I've hit suspicious behavior many times now.
> Looking at the errors Sergey mentions, they do make sense.  We have changed the base
> class of a public class, and I guess the API-tool should have reported a problem.
> It still does not for me.
>
> That being said, the change made is done in such a way that I believe it will not actually
> break the API, because the new base class, is a copy/pasted identical class as the previous
> one.  In that case, I think it would be ok to use an API filter to get rid of the warning.  We may
> need Sergey's help to create the filter, since none of us can see the API error :)
>
> To show the API-tool flakiness, can someone else try this:
> 1- Change the string value of ICDebugConstants.DEF_CHARSET and save.  I get an API-tool error
> 2- Change the string value of ICDebugConstants.PREF_STEP_MODE and save.  I get no error. ???
>
> Thanks
>
> Marc
> ________________________________________
> From: cdt-dev-bounces@xxxxxxxxxxx [cdt-dev-bounces@xxxxxxxxxxx] On Behalf Of Doug Schaefer [cdtdoug@xxxxxxxxx]
> Sent: March 1, 2012 8:56 AM
> To: CDT General developers list.
> Subject: Re: [cdt-dev] API tooling errors
>
> Ah, I mislabelled. I am using 8.0.0. Sergey, what are you using? I've
> been wondering if simply extracting the master zip actually worked
> since it's a p2 repo, not an eclipse install (although they are very
> close).
>
> On Wed, Feb 29, 2012 at 11:21 PM, Mikhail Khodjaiants
> <mikhailkhod@xxxxxxxxxxxxxx> wrote:
>> On 29/02/2012 11:13 PM, Doug Schaefer wrote:
>>
>> I just tried it myself and I don't see anything. I'm using the
>> cdt-master-8.0.2.zip extracted as my baseline (and yes, we should
>> really be using 8.0.0). Is that what others are doing?
>>
>> I am using 8.0.0 as my baseline.
>>
>>
>> _______________________________________________
>> cdt-dev mailing list
>> cdt-dev@xxxxxxxxxxx
>> https://dev.eclipse.org/mailman/listinfo/cdt-dev
>>
> _______________________________________________
> cdt-dev mailing list
> cdt-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/cdt-dev
> _______________________________________________
> cdt-dev mailing list
> cdt-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/cdt-dev
_______________________________________________
cdt-dev mailing list
cdt-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/cdt-dev


Back to the top