problemreporter bug in 0.95.1 released version [message #33416] |
Thu, 16 October 2008 00:47  |
Eclipse User |
|
|
|
Hi,
We are releasing our IDE with DLTK 0.95.1 released version.But there seems
to be a bug in the problem reporter.
The issue is suppose i open a script file, make a small change so as to
produce a syntax error, the errors are reported in problems view.
So far so good; now close the script without saving, the problems view is
not cleared.
When i reopen the file, it opens the saved file which doesnot have any
syntax errors. But my problems view still shows errors of the unsaved
version.
I tried to debug and found that getProblemReporter() in
AbstractSourceElementParser is null, for first parse on the script file,
which is why it fails.
It has been solved in the latest integration build but we are about to
release our product with 0.95.1 dltk released version. Can we have some
fix for this version.
Please do the needful.
Thanks and regards,
sheela.
|
|
|
Re: problemreporter bug in 0.95.1 released version [message #33494 is a reply to message #33416] |
Mon, 20 October 2008 04:16   |
Eclipse User |
|
|
|
This is a multi-part message in MIME format.
--------------050608010501080806080704
Content-Type: text/plain; charset=ISO-8859-15; format=flowed
Content-Transfer-Encoding: 7bit
Hi Sheela,
At the moment you can use a workaround similar to the attached.
We will make the proper fix in CVS and it will be released as part of
the 0.95.2 bugfix release.
Regards,
Alex
Sheela wrote:
> Hi,
> We are releasing our IDE with DLTK 0.95.1 released version.But there
> seems to be a bug in the problem reporter.
>
> The issue is suppose i open a script file, make a small change so as to
> produce a syntax error, the errors are reported in problems view.
>
> So far so good; now close the script without saving, the problems view
> is not cleared.
>
> When i reopen the file, it opens the saved file which doesnot have any
> syntax errors. But my problems view still shows errors of the unsaved
> version.
>
> I tried to debug and found that getProblemReporter() in
> AbstractSourceElementParser is null, for first parse on the script file,
> which is why it fails.
>
> It has been solved in the latest integration build but we are about to
> release our product with 0.95.1 dltk released version. Can we have some
> fix for this version.
>
> Please do the needful.
>
> Thanks and regards,
> sheela.
>
--------------050608010501080806080704
Content-Type: text/plain;
name="reparse-dispose-unsaved-changes.patch"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline;
filename="reparse-dispose-unsaved-changes.patch"
### Eclipse Workspace Patch 1.0
#P org.eclipse.dltk.ruby.ui
Index: src/org/eclipse/dltk/ruby/internal/ui/editor/RubyEditor.java
============================================================ =======
RCS file: /cvsroot/technology/org.eclipse.dltk/ruby/plugins/org.eclips e.dltk.ruby.ui/src/org/eclipse/dltk/ruby/internal/ui/editor/ RubyEditor.java,v
retrieving revision 1.19.4.1
diff -u -r1.19.4.1 RubyEditor.java
--- src/org/eclipse/dltk/ruby/internal/ui/editor/RubyEditor.java 20 Oct 2008 07:45:04 -0000 1.19.4.1
+++ src/org/eclipse/dltk/ruby/internal/ui/editor/RubyEditor.java 20 Oct 2008 08:10:43 -0000
@@ -9,10 +9,17 @@
************************************************************ *******************/
package org.eclipse.dltk.ruby.internal.ui.editor;
+import org.eclipse.core.resources.IFile;
import org.eclipse.core.runtime.CoreException;
import org.eclipse.core.runtime.ListenerList;
+import org.eclipse.dltk.compiler.problem.IProblemFactory;
+import org.eclipse.dltk.core.DLTKLanguageManager;
import org.eclipse.dltk.core.IDLTKLanguageToolkit;
import org.eclipse.dltk.core.IModelElement;
+import org.eclipse.dltk.core.ISourceModule;
+import org.eclipse.dltk.core.ModelException;
+import org.eclipse.dltk.core.SourceParserUtil;
+import org.eclipse.dltk.internal.core.ModelManager;
import org.eclipse.dltk.internal.ui.actions.FoldingActionGroup;
import org.eclipse.dltk.internal.ui.editor.DLTKEditorMessages;
import org.eclipse.dltk.internal.ui.editor.ScriptEditor;
@@ -23,6 +30,7 @@
import org.eclipse.dltk.ruby.internal.ui.text.IRubyPartitions;
import org.eclipse.dltk.ruby.internal.ui.text.RubyPairMatcher;
import org.eclipse.dltk.ruby.internal.ui.text.folding.RubyFoldingSt ructureProvider;
+import org.eclipse.dltk.ui.DLTKUIPlugin;
import org.eclipse.dltk.ui.actions.IScriptEditorActionDefinitionIds ;
import org.eclipse.dltk.ui.text.ScriptTextTools;
import org.eclipse.dltk.ui.text.folding.IFoldingStructureProvider;
@@ -195,6 +203,27 @@
if (sourceViewer instanceof ITextViewerExtension)
((ITextViewerExtension) sourceViewer)
.removeVerifyKeyListener(fBracketInserter);
+ if (isDirty()) {
+ final IModelElement modelElement = getInputModelElement();
+ if (modelElement != null) {
+ ISourceModule module = (ISourceModule) modelElement
+ .getAncestor(IModelElement.SOURCE_MODULE);
+ ModelManager.getModelManager().getSourceModuleInfoCache()
+ .remove(module);
+ IFile resource = (IFile) module.getResource();
+ try {
+ module.getBuffer().setContents(
+ org.eclipse.dltk.internal.core.util.Util
+ .getResourceContentsAsCharArray(resource));
+ } catch (ModelException e) {
+ DLTKUIPlugin.log(e);
+ }
+ IProblemFactory factory = DLTKLanguageManager
+ .getProblemFactory(getLanguageToolkit().getNatureId());
+ SourceParserUtil.getModuleDeclaration(module, factory
+ .createReporter(resource));
+ }
+ }
super.dispose();
}
--------------050608010501080806080704--
|
|
|
Re: problemreporter bug in 0.95.1 released version [message #33527 is a reply to message #33494] |
Mon, 20 October 2008 04:49  |
Eclipse User |
|
|
|
Sorry, probably it is not clear from the patch: this code should be
placed in the dispose() method of your editor extending ScriptEditor.
Alex Panchenko wrote:
> Hi Sheela,
>
> At the moment you can use a workaround similar to the attached.
>
> We will make the proper fix in CVS and it will be released as part of
> the 0.95.2 bugfix release.
>
> Regards,
> Alex
>
>
> Sheela wrote:
>> Hi,
>> We are releasing our IDE with DLTK 0.95.1 released version.But there
>> seems to be a bug in the problem reporter.
>>
>> The issue is suppose i open a script file, make a small change so as
>> to produce a syntax error, the errors are reported in problems view.
>>
>> So far so good; now close the script without saving, the problems view
>> is not cleared.
>>
>> When i reopen the file, it opens the saved file which doesnot have any
>> syntax errors. But my problems view still shows errors of the unsaved
>> version.
>>
>> I tried to debug and found that getProblemReporter() in
>> AbstractSourceElementParser is null, for first parse on the script
>> file, which is why it fails.
>>
>> It has been solved in the latest integration build but we are about to
>> release our product with 0.95.1 dltk released version. Can we have
>> some fix for this version.
>>
>> Please do the needful.
>>
>> Thanks and regards,
>> sheela.
>>
>
|
|
|
Powered by
FUDForum. Page generated in 0.03638 seconds