Attachment 151966 Details for Bug 294887 – Patch to fix the EOF

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] Patch to fix the EOF

patch_wtp31.txt (text/plain), 1.74 KB, created by Wini Mark

on 2009-11-11 14:33:26 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Wini Mark

Created: 2009-11-11 14:33:26 EST

Size: 1.74 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.wst.validation
>Index: vf2/org/eclipse/wst/validation/internal/DependencyIndex.java
>===================================================================
>RCS file: /cvsroot/webtools/common/plugins/org.eclipse.wst.validation/vf2/org/eclipse/wst/validation/internal/DependencyIndex.java,v
>retrieving revision 1.13
>diff -u -r1.13 DependencyIndex.java
>--- vf2/org/eclipse/wst/validation/internal/DependencyIndex.java	3 Apr 2009 12:26:36 -0000	1.13
>+++ vf2/org/eclipse/wst/validation/internal/DependencyIndex.java	11 Nov 2009 19:28:27 -0000
>@@ -12,6 +12,7 @@
> 
> import java.io.DataInputStream;
> import java.io.DataOutputStream;
>+import java.io.EOFException;
> import java.io.File;
> import java.io.FileInputStream;
> import java.io.FileOutputStream;
>@@ -155,6 +156,9 @@
> 					}					
> 				}				
> 			}
>+			catch (EOFException e){
>+				Tracing.log("Unable to read the dependency index file because of EOF exception"); 
>+			}
> 			catch (IOException e){
> 				error = true;
> 				ValidationPlugin.getPlugin().handleException(e);
>@@ -246,10 +250,11 @@
> 	public synchronized void saving(ISaveContext context) throws CoreException {
> 		if (!_dirty)return;
> 		_dirty = false;
>-		
>+		boolean error = false;
> 		DataOutputStream out = null;
>+		File f = null;
> 		try {
>-			File f = getIndexLocation();
>+			f = getIndexLocation();
> 			out = new DataOutputStream(new FileOutputStream(f));
> 			out.writeInt(CurrentVersion);
> 			Map<String, Set<DependsResolved>> map = compress(_dependsOn);
>@@ -268,10 +273,12 @@
> 			}
> 		}
> 		catch (IOException e){
>+			error = true;
> 			ValidationPlugin.getPlugin().handleException(e);
> 		}
>-		finally {
>+		finally {		
> 			Misc.close(out);
>+			if (error)f.delete();
> 		}
> 	}
>

Actions: View | Diff

Attachments on bug 294887: 151966