RE: [cdt-patch] Patch proposal for Bug 43174

[Thomas Fletcher <ThomasF@xxxxxxx>]

> -----Original Message-----
> From: Alain Magloire [mailto:alain@xxxxxxx]
> Sent: October 8, 2003 10:55 AM
> To: cdt-patch@xxxxxxxxxxx
> Subject: Re: [cdt-patch] Patch proposal for Bug 43174
> 
> +1, with some comments.
> 
> (1)
> If the file search scope includes the inclusion ("#include ...") 
> part of the search, got no problem.
> True for C it is not a requirement for visibility, in the IDE
> case making an explicit declaration should be a requirement for
> Code Assist to work properly.

Fully agree with this.  One of the major feature parity issues 
which customers (of the CDT and derived products) have is the 
fact that we _do_ have code completion for projects.  

I can certainly explain to existing customers about the necessity
(ie performance improvement) of only providing information from 
included header files (promote proper developer discipline, clean
code yadadada).  I can _not_ realistically tell them that the entire
feature is _just_gone_ and that they will have to live with it.
 
> (2)
> I do understand that the first time, you do content assit
> you will have a hit, but do not we have cache indexed file
> to speed things up ?
> 
> (3)
> The old indexer(CTags) parse the entire project once
> in a database, and all globals would be indexed so
> next time around even for a big project, it would not
> take more then a few seconds.

I think that there is likely some extra work being done here that
Alex Chapiro is discussing in another thread.

Thanks,
 Thomas


> > JohnC
> > 
> > cdt-patch-admin@xxxxxxxxxxx wrote on 10/08/2003 05:54:36 AM:
> > 
> > > 
> > > What's the problem? 
> > > 
> > > In its current state, code assist takes a long time to pop up 
> > > completions and they aren't always correct. 
> > > 
> > > Why? 
> > > 
> > > Code assist  currently uses a project scope when performing a 
> > > search. Thus all files in a project are searched - which 
> > > contributes to the long pop up time (especially in large 
> projects) 
> > > and the inaccurate results (as a file rarely includes all 
> > > the other files in the same project). 
> > > 
> > > What's in this patch? 
> > > 
> > > This patch changes the search scope for code assist from 
> a project 
> > > scope to a file scope. This improves both performance and 
> > > accuracy of the matches. 
> > > 
> > > Automation Suite run on both Windows + Linux. 
> > > 
> > > Thanks, 
> > > Bogdan 
> > > [attachment "cdt-core-patch.txt" deleted by John 
> Camelon/Ottawa/IBM]
> > > [attachment "cdt-ui-patch.txt" deleted by John 
> Camelon/Ottawa/IBM] 
> > _______________________________________________
> > cdt-patch mailing list
> > cdt-patch@xxxxxxxxxxx
> > http://dev.eclipse.org/mailman/listinfo/cdt-patch
> > 
> > 
> > _______________________________________________
> > cdt-patch mailing list
> > cdt-patch@xxxxxxxxxxx
> > http://dev.eclipse.org/mailman/listinfo/cdt-patch
> > 
> 
> 
> -- 
> au revoir, alain
> ----
> Aussi haut que l'on soit assis, on est toujours assis que sur 
> son cul !!!
> 
> _______________________________________________
> cdt-patch mailing list
> cdt-patch@xxxxxxxxxxx
> http://dev.eclipse.org/mailman/listinfo/cdt-patch
>