[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
Re: [cdt-dev] CODAN-Auto checking after quickfix
|
Alena Laskavaia <elaskavaia.cdt@xxxxxxxxx> a écrit :
On Tue, Apr 12, 2011 at 10:08 AM, <maximed@xxxxxxxxxxxxxxxxxxx> wrote:
Hi.
I noticed that whe I used a quickfix on my code, checkers were called
automatically to refresh errors and warnings. I'd like to know if there is a
way to do it without modifying my file or my AST (I am in my justification
thing again).
I don't understand the question
When a codan problem is reproted on CDT IDE, and when I use the
suggested quick fix (such as create local variable for example), the
error/warning disappears, which is the expected comportment. Indeed,
in the existing quickfixes, the source file is modified, so I think
the warning disappears because of the modification (as I am in run as
you type mode). The quickfix I developed does'nt modify the source to
change the problem, it only flags it in an other way. My problem is
that I need to add/suppress a character (in run as you type mode), or
launch a manual checking to make this modification visible (ie re-call
checking on the file). So I wanted to know what was the way (the
class/method) to explicitly call checking of all my checkers on the
displayed file.
My second question concerns the fact that when I use a quickfix, a new tab
is opened in the editor, and sometimes eclipse asks me to save something. Is
it because I used "run as eclipse application" (and then it will not be the
case when I'll run it it production mode), or because something is wrong in
my installation?
It should not open tab. It will ask to save the file because it cannot
work on unsaved file for some reason.
Check the code of existing quick fixes to see what is different.
The fact is that my code is based on a copy/paste of what was done on
previous quickfixes (I don't pretend being smart enough to guess what
API does without an example =) ). Even with an empty apply method, the
problem is present. Maybe it comes from my build, as I had to manually
copy a class from cdt.core into codan.checkers.ui, since I wasn't able
to find it in the cdt.core.jar which was in my eclipse/plugin
directory (don't remember at the moment the name of the class). The
weird thing is that it asks to save the file, saves it indeed, but the
result file is empty...
Here is my code for sample:
/**
* Handles the quick fix
*/
@Override
public void apply(IMarker pMarker, IDocument pDocument) {
try {
/* We get the beginning and the ending of the problem
*/
int lStart=(Integer)
pMarker.getAttribute(IMarker.CHAR_START);
int lEnd=(Integer)
pMarker.getAttribute(IMarker.CHAR_END);
/* We get the AST of the program, so as to get its
filename */
ITranslationUnit
lUnit=getTranslationUnitViaEditor(pMarker);
IASTTranslationUnit lAst = lUnit.getAST();
String lFilename=lAst.getContainingFilename();
/* We get the id of the problem, so as to get its
checker's id */
String lProblemID=getProblemId(pMarker);
/* If the problem is an error or a warning, it is not
justified yet */
Integer
lSeverity=(Integer)pMarker.getAttribute(IMarker.SEVERITY);
IASTNodeSelector
lSelector=lAst.getNodeSelector(lFilename);
int lLength=lEnd-lStart;
IASTNode lNode=lSelector.findEnclosingNode(lStart,
lLength);
String lExpression=lNode.getRawSignature();
if(lSeverity==IMarker.SEVERITY_ERROR ||
lSeverity==IMarker.SEVERITY_WARNING){
/* Ables to enter justification */
InputDialog lDialog = new InputDialog(null,
"Justification",
"Please enter the
justification", "Default", null);
lDialog.open();
String lReason = lDialog.getValue();
CodanCheckersActivator.getDefault().GDatabase.addJustification(lFilename,
lProblemID, lStart, lEnd, lExpression, lReason);
}
/* Else, if it is an info, it is already justified */
else{
CodanCheckersActivator.getDefault().GDatabase.removeJustification(lFilename,
lProblemID, lStart, lEnd, lExpression);
}
} catch (CoreException e) {
e.printStackTrace();}
}
Thank in advance.
Maxime
_______________________________________________
cdt-dev mailing list
cdt-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/cdt-dev