RE: [cdt-debug-dev] patch for 80716- Global arrays show only the first e

["Alexiev, Dobrin" <dalexiev@xxxxxx>]

Thanks Alan, 

-----Original Message-----
From: cdt-debug-dev-admin@xxxxxxxxxxx
[mailto:cdt-debug-dev-admin@xxxxxxxxxxx] On Behalf Of Alain Magloire
Sent: Friday, December 10, 2004 5:04 PM
To: cdt-debug-dev@xxxxxxxxxxx
Subject: Re: [cdt-debug-dev] patch for 80716- Global arrays show only
the first element in the Variable View

> 
> Hi,=20
> During the last 6 months we at Texas Instruments were developing
product
> based on CDT. During that period we were making fixed, enhancement and
> changes in CDT that would like to share with the rest of the
community.=20
> 

This is excellent news !!
More feedbacks and resources are always appreciated.

> Since our beta product is out we have more time to start submitting
the
> patches and enhancements we looked at. Also I would like to share ours
> customer suggestions and enter them as enhancements.=20
> 

Your Beta is based on which branch?  we have different branches
CDT_1_2_1, CDT_2_0, CDT_21

Currently the "active" streams are CDT_21 and the head stream,
an the CDT_21 is coming on its final days.

>>>>>>> Our beta is based on CDT_2_0_2. When I entered the PR I
specified it is for CDT 2_0_2. I checked in the HEAD but the class has
changed so I wasn't sure if the problem was still or not. I left this to
the committer since at this moment I didn't have a chance to get
familiar with the delta between CDT_2_0_2 and HEAD. 

> I'll start with filing "80716 - Global arrays show only the first
> element in the Variable View" and adding a patch for it.=20
> 

We've tried in the pass to accomodate ISV, by applying patches when
necessary.
It is really on case by case(i.e. if the patch does not rewrite
the entire plugin 8-), folks here been very open.
So when creating the PR it should be stated upfront to which branch and
if it also valid for the head.

>>>>>>> I specified the bug is for CDT_2_0_2. Next time I will specify
if it is applicable for the HEAD. In the meantime I hope I will have
more chance to get familiar with the HEAD. 

> I would also like to create a test case for that bug.=20
> Any pointers where and how to add a test case for that one?
> 

Test cases ... sigh ... We started a JUnit plugin for the debug core/ui
It is something that we are looking for CDT-3.0.
The debug{core,ui} lack regression testing, a weak spot.

So yes, your efforts are more then welcome
see org.eclipse.cdt.debug.ui.tests

>>>> Sure. I see if there are similar test cases and I will add one for
this PR. I already started looking at this test plugin. I have compared
I have the "test cases" item on my "to do" list and I think this one is
one of the first one to write. 

_______________________________________________
cdt-debug-dev mailing list
cdt-debug-dev@xxxxxxxxxxx
http://dev.eclipse.org/mailman/listinfo/cdt-debug-dev