Re: [cdt-patch] Support for task tags, and more... (initial revision)

[John Camelon <jcamelon@xxxxxxxxxx>]

Also ... how does this correspond to
real compiler build errors in the task view?  Are we to consolidate
build errors w/our own IProblems so that the user does not receive 2 tasks
for every real problem?

JohnC

John Camelon/Ottawa/IBM@IBMCA
Sent by: cdt-patch-admin@xxxxxxxxxxx

06/27/2003 10:00 AM

Please respond to
cdt-patch@xxxxxxxxxxx

To	cdt-patch@xxxxxxxxxxx
cc
Subject	Re: [cdt-patch] Support for task tags, and more... (initial revision)

Cool Victor!  Thanks for getting IProblem started, I've been dreading
that.   I have no problem with the work done on the Parser side, I'll
let Alain comment on the C-Model/UI portion and leave it in his hands to
apply the patch.   

I do have a question about the overall architecture to support this : is
it the indexer that is expected to update the TranslationUnit problem list?
 The CModel will never do a full parse and thus will not catch any
problems inside of method bodies ... but the indexer is doing the full
parse on all translation unit/build property changes.  

JohnC

"Victor Mozgin" <vic_mozgin@xxxxxxxxxxx>
Sent by: cdt-patch-admin@xxxxxxxxxxx

06/27/2003 05:38 AM

Please respond to
cdt-patch@xxxxxxxxxxx

To	<cdt-patch@xxxxxxxxxxx>
cc
Subject	Re: [cdt-patch] Support for task tags, and more... (initial revision)

        Here it is. Please note that there is a new
directory - org.eclipse.cdt.core\parser\org\eclipse\cdt\internal\core\parser\problem,
you might need to do manual "cvs add" on it.
  
            /Vic 
----- Original Message ----- 
From: John
Camelon 
To: cdt-patch@xxxxxxxxxxx
Sent: Friday, June 27, 2003 2:46 AM 
Subject: Re: [cdt-patch] Support for task tags, and more... (initial
revision) 


Hi Victor, 

Could you do a CVS update and resend the patch?  
Sorry to keep checking stuff in but Kleo here has me on a tight schedule
.. :-) 

JohnC

"Victor Mozgin" <vic_mozgin@xxxxxxxxxxx>
Sent by: cdt-patch-admin@xxxxxxxxxxx

06/26/2003 01:10 PM

Please respond to
cdt-patch@xxxxxxxxxxx

To	<cdt-patch@xxxxxxxxxxx>
cc
Subject	[cdt-patch] Support for task tags, and more... (initial revision)

    This massive patch started as an attempt to port JDT task
tags support
to CDT. Along the way I dragged significant parts of JDT infrastructure
dealing with options/preferences handling, and problem reporting in general.
This should simplify extension of CDT parser/model builder with error
reporting capabilities similar to JDT (with tasks and annotations in
editors).
   As it is getting difficult to keep in sync with the core trunk,
I
decided to submit this patch now. Core part should be mostly finished;
there
are still some known issues with the UI part (I'm working on it):
   - task tag highlighting in editors only takes into account plug-in
level preferences, and ignores project level ones, even if latter override;
   - preference listeners do not work, hence all the changes in
preferences will be visible after next reparse;
   - changes in task tags preferences will trigger a request to rebuild
the project (JDT-style), while in case of CDT it should do something like
reparse (?)

This should work:
   - task tags in task panel, editor annotations (left and right),
with
proper location information;
   - doesn't create duplicate tasks and removes obsolete (i.e. removed
from the code) tasks (something that JDT doesn't do for me!)
   - persistent plug-in level preferences for task tags and priorities,
and text style for task tags highlighting;
   - persistent project level preferences for task tags and priorities
(may override);

             /Vic
[attachment "ui20030626.patch" deleted by John Camelon/Ottawa/IBM]
[attachment "core20030626.patch" deleted by John Camelon/Ottawa/IBM]
[attachment "ui20030627.patch" deleted by John Camelon/Ottawa/IBM]
[attachment "core20030627.patch" deleted by John Camelon/Ottawa/IBM]