Attachment 144827 Details for Bug 144810 – Fix Patch

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

[patch] Fix Patch

144810_erroneous_jsp_directives.txt (text/plain), 1.31 KB, created by Ian Tewksbury

on 2009-08-18 10:29:12 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Ian Tewksbury

Created: 2009-08-18 10:29:12 EDT

Size: 1.31 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.jst.jsp.core
>Index: src/org/eclipse/jst/jsp/core/internal/parser/JSPSourceParser.java
>===================================================================
>RCS file: /cvsroot/webtools/sourceediting/plugins/org.eclipse.jst.jsp.core/src/org/eclipse/jst/jsp/core/internal/parser/JSPSourceParser.java,v
>retrieving revision 1.16
>diff -u -r1.16 JSPSourceParser.java
>--- src/org/eclipse/jst/jsp/core/internal/parser/JSPSourceParser.java	10 Apr 2007 17:02:55 -0000	1.16
>+++ src/org/eclipse/jst/jsp/core/internal/parser/JSPSourceParser.java	18 Aug 2009 14:20:57 -0000
>@@ -360,6 +360,13 @@
> 				if (currentNode.getLastRegion() != null && currentNode.getLastRegion().getType() == DOMRegionContext.UNDEFINED) {
> 					currentNode.getLastRegion().adjustLength(region.getLength());
> 					currentNode.adjustLength(region.getLength());
>+					
>+					//if adding this region to a previous container then need to add this
>+					//region to the container and update its start location
>+					if(currentNode.getLastRegion() instanceof ITextRegionContainer) {
>+						region.adjustStart(-currentNode.getLastRegion().getStart()- currentNode.getStart());
>+						((ITextRegionContainer)currentNode.getLastRegion()).getRegions().add(region);
>+					}
> 				}
> 				// previous wasn't undefined
> 				else {

Flags:

nsand.dev: iplog+

Actions: View | Diff

Attachments on bug 144810: 144827