Attachment 209487 Details for Bug 365541 – Updated patch.

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

[patch] Updated patch.

LargeArrays.patch (text/plain), 29.00 KB, created by Nobody - feel free to take it

on 2012-01-13 17:48:46 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Nobody - feel free to take it

Created: 2012-01-13 17:48:46 EST

Size: 29.00 KB

patch

obsolete

>From e5569308a1e02fdbc706252eeeefb9d30b0ef4e2 Fri, 13 Jan 2012 17:23:15 -0500
>From: Mikhail Khodjaiants <mikhailkhod@googlemail.com>
>Date: Fri, 13 Jan 2012 17:21:31 -0500
>Subject: [PATCH] Bug 365541 - View an array variable whose length is very long ï¼such as 10000ï¼ in the editor or Variable View, will cause the interface to die.
>
>diff --git a/debug/org.eclipse.cdt.debug.ui/icons/obj16/arraypartition_obj.gif b/debug/org.eclipse.cdt.debug.ui/icons/obj16/arraypartition_obj.gif
>new file mode 100755
>index 0000000..052915a
>--- /dev/null
>+++ b/debug/org.eclipse.cdt.debug.ui/icons/obj16/arraypartition_obj.gif
>Binary files differ
>diff --git a/debug/org.eclipse.cdt.debug.ui/src/org/eclipse/cdt/debug/internal/ui/CDebugImages.java b/debug/org.eclipse.cdt.debug.ui/src/org/eclipse/cdt/debug/internal/ui/CDebugImages.java
>index 5387b5e..26c89c6 100644
>--- a/debug/org.eclipse.cdt.debug.ui/src/org/eclipse/cdt/debug/internal/ui/CDebugImages.java
>+++ b/debug/org.eclipse.cdt.debug.ui/src/org/eclipse/cdt/debug/internal/ui/CDebugImages.java
>@@ -104,6 +104,7 @@
> 	public static final String IMG_OBJS_PATH_MAPPING = NAME_PREFIX + "mapping_obj.gif";	//$NON-NLS-1$
> 	public static final String IMG_OBJS_PATH_MAP_ENTRY = NAME_PREFIX + "mapentry_obj.gif";	//$NON-NLS-1$
> 	public static final String IMG_OBJS_COMMON_TAB = NAME_PREFIX + "common_tab.gif"; //$NON-NLS-1$
>+	public static final String IMG_OBJS_ARRAY_PARTITION = NAME_PREFIX + "arraypartition_obj.gif"; //$NON-NLS-1$
> 
> 	public static final String IMG_LCL_TYPE_NAMES = NAME_PREFIX + "tnames_co.gif";	//$NON-NLS-1$
> 	public static final String IMG_LCL_CHANGE_REGISTER_VALUE = NAME_PREFIX + "change_reg_value_co.gif";	//$NON-NLS-1$
>@@ -188,6 +189,7 @@
> 	public static final ImageDescriptor DESC_OBJS_PATH_MAPPING = createManaged(T_OBJ, IMG_OBJS_PATH_MAPPING);
>     public static final ImageDescriptor DESC_OBJS_PATH_MAP_ENTRY = createManaged(T_OBJ, IMG_OBJS_PATH_MAP_ENTRY);
>     public static final ImageDescriptor DESC_OBJS_COMMON_TAB = createManaged(T_OBJ, IMG_OBJS_COMMON_TAB);
>+    public static final ImageDescriptor DESC_OBJS_ARRAY_PARTITION = createManaged(T_OBJ, IMG_OBJS_ARRAY_PARTITION);
> 	public static final ImageDescriptor DESC_WIZBAN_ADD_SOURCE = createManaged(T_WIZBAN, IMG_WIZBAN_ADD_SOURCE);
> 	public static final ImageDescriptor DESC_WIZBAN_PATH_MAPPING = createManaged(T_WIZBAN, IMG_WIZBAN_PATH_MAPPING);
> 	public static final ImageDescriptor DESC_WIZBAN_PATH_MAP_ENTRY = createManaged(T_WIZBAN, IMG_WIZBAN_PATH_MAP_ENTRY);
>diff --git a/dsf-gdb/org.eclipse.cdt.dsf.gdb/src/org/eclipse/cdt/dsf/mi/service/MIExpressions.java b/dsf-gdb/org.eclipse.cdt.dsf.gdb/src/org/eclipse/cdt/dsf/mi/service/MIExpressions.java
>index 9f978a8..34ab647 100644
>--- a/dsf-gdb/org.eclipse.cdt.dsf.gdb/src/org/eclipse/cdt/dsf/mi/service/MIExpressions.java
>+++ b/dsf-gdb/org.eclipse.cdt.dsf.gdb/src/org/eclipse/cdt/dsf/mi/service/MIExpressions.java
>@@ -13,12 +13,14 @@
>  *******************************************************************************/
> package org.eclipse.cdt.dsf.mi.service;
> 
>+import java.util.Arrays;
> import java.util.HashMap;
> import java.util.Hashtable;
> import java.util.Map;
> 
> import org.eclipse.cdt.core.IAddress;
> import org.eclipse.cdt.dsf.concurrent.DataRequestMonitor;
>+import org.eclipse.cdt.dsf.concurrent.ImmediateExecutor;
> import org.eclipse.cdt.dsf.concurrent.ImmediateRequestMonitor;
> import org.eclipse.cdt.dsf.concurrent.RequestMonitor;
> import org.eclipse.cdt.dsf.datamodel.AbstractDMContext;
>@@ -70,6 +72,8 @@
>  * @since 2.0
>  */
> public class MIExpressions extends AbstractDsfService implements IMIExpressions, ICachingService {
>+
>+	private static final int PARTITION_LENGTH = 100;
> 
>     /**
>      * A format that gives more details about an expression and supports pretty-printing
>@@ -381,7 +385,7 @@
> 			this.exprInfo = info;
> 		}
>     }
>-    
>+
>     protected static class InvalidContextExpressionDMC extends AbstractDMContext 
>         implements IExpressionDMContext
>     {
>@@ -411,6 +415,85 @@
>     	@Override
>         public String getExpression() {
>             return expression;
>+        }
>+    }
>+
>+    /**
>+	 * @since 4.1
>+	 */
>+    protected static class IndexedPartitionDMC extends MIExpressionDMC implements IIndexedPartitionDMContext {
>+
>+    	final private ExpressionInfo fParentInfo;
>+		private final int fIndex;
>+    	private final int fLength;
>+    	
>+		public IndexedPartitionDMC(
>+				String sessionId, 
>+				ExpressionInfo parentInfo, 
>+				IFrameDMContext frameCtx, 
>+				int index, 
>+				int length) {
>+			this(sessionId, parentInfo, (IDMContext)frameCtx, index, length);
>+		}
>+
>+    	private IndexedPartitionDMC(
>+    			String sessionId, 
>+    			ExpressionInfo parentInfo, 
>+    			IDMContext parent, 
>+    			int index, 
>+    			int length) {
>+			super(sessionId, createExpressionInfo(parentInfo, index, length), parent);
>+			fIndex = index;
>+			fLength = length;
>+			fParentInfo = parentInfo;
>+		}
>+
>+    	public ExpressionInfo getParentInfo() {
>+    		return fParentInfo;
>+    	}
>+
>+		/* (non-Javadoc)
>+		 * @see org.eclipse.cdt.dsf.debug.service.IExpressions4.IIndexedPartitionDMContext#getParentExpression()
>+		 */
>+		@Override
>+		public String getParentExpression() {
>+			return getParentInfo().getFullExpr();
>+		}
>+
>+		@Override
>+		public int getIndex() {
>+			return fIndex;
>+		}
>+
>+		@Override
>+		public int getLength() {
>+			return fLength;
>+		}
>+
>+        @Override
>+        public boolean equals(Object other) {
>+            return super.baseEquals(other) && 
>+                    ((IndexedPartitionDMC) other).getIndex() == getIndex() &&
>+                    ((IndexedPartitionDMC) other).getLength() == getLength();                
>+        }
>+
>+        @Override
>+        public int hashCode() {
>+            return super.baseHashCode() + 17*getIndex() + 31*getLength();
>+        }
>+
>+        @Override
>+        public String toString() {
>+            return String.format( "%s[%d-%d]", baseToString(), Integer.valueOf( getIndex() ), Integer.valueOf( getIndex() + getLength() ) ); //$NON-NLS-1$
>+        }
>+
>+        private static ExpressionInfo createExpressionInfo(ExpressionInfo parentInfo, int index, int length) {
>+        	String expression = String.format(
>+        			"*((%s)+%d)@%d",  //$NON-NLS-1$
>+        			parentInfo.getFullExpr(), 
>+        			Integer.valueOf(index), 
>+        			Integer.valueOf(length));
>+        	return new ExpressionInfo(expression, expression);
>         }
>     }
>     
>@@ -1064,33 +1147,8 @@
> 	 *            The data request monitor that will contain the requested data
> 	 */
> 	@Override
>-	public void getSubExpressions(final IExpressionDMContext dmc,
>-			final DataRequestMonitor<IExpressionDMContext[]> rm) 
>-	{		
>-		if (dmc instanceof MIExpressionDMC) {
>-			fExpressionCache.execute(
>-					new ExprMetaGetChildren(dmc),				
>-					new DataRequestMonitor<ExprMetaGetChildrenInfo>(getExecutor(), rm) {
>-						@Override
>-						protected void handleSuccess() {
>-							ExpressionInfo[] childrenExpr = getData().getChildrenExpressions();
>-							IExpressionDMContext[] childArray = new IExpressionDMContext[childrenExpr.length];
>-							for (int i=0; i<childArray.length; i++) {
>-								childArray[i] = createExpression(
>-										dmc.getParents()[0], childrenExpr[i]);
>-							}
>-
>-							rm.setData(childArray);
>-							rm.done();
>-						}
>-					});
>-		} else if (dmc instanceof InvalidContextExpressionDMC) {
>-			rm.setStatus(new Status(IStatus.ERROR, GdbPlugin.PLUGIN_ID, INVALID_HANDLE, "Invalid context for evaluating expressions.", null)); //$NON-NLS-1$
>-			rm.done();
>-		} else {
>-			rm.setStatus(new Status(IStatus.ERROR, GdbPlugin.PLUGIN_ID, INTERNAL_ERROR, "Invalid expression context.", null)); //$NON-NLS-1$
>-			rm.done();
>-		}
>+	public void getSubExpressions(IExpressionDMContext dmc, DataRequestMonitor<IExpressionDMContext[]> rm) {
>+		getSubExpressions(dmc, -1, -1, rm);
> 	}
> 
> 	/**
>@@ -1112,37 +1170,42 @@
> 	public void getSubExpressions(final IExpressionDMContext exprCtx, final int startIndex,
> 			final int length, final DataRequestMonitor<IExpressionDMContext[]> rm) {
> 
>-		if (startIndex < 0 || length < 0) {
>-			rm.setStatus(new Status(IStatus.ERROR, GdbPlugin.PLUGIN_ID, INTERNAL_ERROR, "Invalid range for evaluating sub expressions.", null)); //$NON-NLS-1$
>-			rm.done();
>-			return;
>+		if (exprCtx instanceof IndexedPartitionDMC) {
>+			getIndexedPartitionChildren((IndexedPartitionDMC)exprCtx, startIndex, length, rm);
> 		}
>-		
>-		if (exprCtx instanceof MIExpressionDMC) {
>-			fExpressionCache.execute(
>-					new ExprMetaGetChildren(exprCtx, startIndex + length),				
>-					new DataRequestMonitor<ExprMetaGetChildrenInfo>(getExecutor(), rm) {
>-						@Override
>-						protected void handleSuccess() {
>-							ExpressionInfo[] childrenExpr = getData().getChildrenExpressions();
>-
>-							if (startIndex >= childrenExpr.length) {
>-								rm.setStatus(new Status(IStatus.ERROR, GdbPlugin.PLUGIN_ID, REQUEST_FAILED, "Invalid range for evaluating sub expressions.", null)); //$NON-NLS-1$
>-								rm.done();
>-								return;
>-							}
>-
>-							int numChildren = childrenExpr.length - startIndex;
>-							numChildren = Math.min(length, numChildren);
>-							IExpressionDMContext[] childrenArray = new IExpressionDMContext[numChildren];
>-							for (int i=0; i < numChildren; i++) {
>-								childrenArray[i] = createExpression(
>-										exprCtx.getParents()[0], childrenExpr[startIndex + i]);
>-							}
>-							rm.setData(childrenArray);
>-							rm.done();
>+		else if (exprCtx instanceof MIExpressionDMC) {
>+			getRealSubExpressionCount(
>+				exprCtx, 
>+				IMIExpressions.CHILD_COUNT_LIMIT_UNSPECIFIED, 
>+				new DataRequestMonitor<Integer>(getExecutor(), rm) {
>+					/* (non-Javadoc)
>+					 * @see org.eclipse.cdt.dsf.concurrent.RequestMonitor#handleSuccess()
>+					 */
>+					@Override
>+					protected void handleSuccess() {
>+						final int realNumChildren = getData().intValue();
>+						if (realNumChildren <= PARTITION_LENGTH) {
>+							getRealSubExpressions(exprCtx, startIndex, length, rm);
> 						}
>-					});
>+						else {
>+							getExpressionData(
>+									exprCtx, 
>+									new DataRequestMonitor<IExpressionDMData>(ImmediateExecutor.getInstance(), rm) {
>+
>+										@Override
>+										protected void handleSuccess() {
>+											if (IExpressionDMData.BasicType.array.equals(getData().getBasicType())) {
>+												rm.setData(getTopLevelIndexedPartitions((MIExpressionDMC)exprCtx, realNumChildren, startIndex, length ));
>+												rm.done();
>+											}
>+											else {
>+												getRealSubExpressions(exprCtx, startIndex, length, rm);
>+											}
>+										}
>+									});
>+						}
>+					}
>+				});
> 		} else if (exprCtx instanceof InvalidContextExpressionDMC) {
> 			rm.setStatus(new Status(IStatus.ERROR, GdbPlugin.PLUGIN_ID, INVALID_HANDLE, "Invalid context for evaluating expressions.", null)); //$NON-NLS-1$
> 			rm.done();
>@@ -1183,19 +1246,48 @@
> 	 * @since 4.0
> 	 */
> 	@Override
>-	public void getSubExpressionCount(IExpressionDMContext dmc,
>+	public void getSubExpressionCount(final IExpressionDMContext dmc,
> 			final int numChildLimit, final DataRequestMonitor<Integer> rm) {
> 
> 		if (dmc instanceof MIExpressionDMC) {
>-			fExpressionCache.execute(
>-					new ExprMetaGetChildCount(dmc, numChildLimit),				
>-					new DataRequestMonitor<ExprMetaGetChildCountInfo>(getExecutor(), rm) {
>+			if (dmc instanceof IndexedPartitionDMC) {
>+				int length = ((IndexedPartitionDMC)dmc).getLength();
>+				rm.setData(computeNumberOfChildren(length));
>+				rm.done();
>+			}
>+			else {
>+				getRealSubExpressionCount( 
>+					dmc, 
>+					numChildLimit, 
>+					new DataRequestMonitor<Integer>(getExecutor(), rm) {
>+
> 						@Override
> 						protected void handleSuccess() {
>-							rm.setData(getData().getChildNum());
>-							rm.done();
>-						}
>+							final int realNum = getData().intValue();
>+							if (realNum < PARTITION_LENGTH) {
>+								rm.setData(Integer.valueOf(realNum));
>+								rm.done();
>+							}
>+							else {
>+								getExpressionData(
>+									dmc, 
>+									new DataRequestMonitor<IExpressionDMData>(ImmediateExecutor.getInstance(), rm) {
>+
>+										@Override
>+										protected void handleSuccess() {
>+											if (IExpressionDMData.BasicType.array.equals(getData().getBasicType())) {
>+												rm.setData(computeNumberOfChildren(realNum));
>+											}
>+											else {
>+												rm.setData(Integer.valueOf(realNum));
>+											}
>+											rm.done();
>+										}
>+									});
>+							}
>+						}				
> 					});
>+			}
> 		} else if (dmc instanceof InvalidContextExpressionDMC) {
> 			rm.setStatus(new Status(IStatus.ERROR, GdbPlugin.PLUGIN_ID, INVALID_HANDLE, "Invalid context for evaluating expressions.", null)); //$NON-NLS-1$
> 			rm.done();
>@@ -1442,4 +1534,178 @@
>             }
>         }); 
>     }
>+
>+	private IndexedPartitionDMC[] getTopLevelIndexedPartitions(
>+			MIExpressionDMC exprCtx, 
>+			int realNumChildren, 
>+			int startIndex, 
>+			int length) {
>+
>+		int numChildren = computeNumberOfChildren(realNumChildren);
>+		if (startIndex >= numChildren)
>+			return new IndexedPartitionDMC[0];
>+		int startIndex1 = (startIndex < 0) ? 0 : startIndex;
>+		int length1 = (length < 0) ? numChildren - startIndex1 : Math.min(length, numChildren - startIndex1);
>+
>+		IndexedPartitionDMC[] children = new IndexedPartitionDMC[numChildren];
>+		int index = 0;
>+		for(int i = 0; i < children.length; ++i) {
>+			int partLength = computePartitionLength(realNumChildren, i);
>+			children[i] = createIndexedPartition(
>+				exprCtx.getParents()[0], 
>+				exprCtx.getExpressionInfo(), 
>+				index, 
>+				partLength);
>+			index += partLength;
>+		}
>+		return Arrays.copyOfRange(children, startIndex1, startIndex1 + length1 );
>+	}
>+
>+	private void getIndexedPartitionChildren(
>+			final IndexedPartitionDMC partDmc, 
>+			final int startIndex,
>+			final int length, 
>+			final DataRequestMonitor<IExpressionDMContext[]> rm) {
>+		
>+		final int startIndex1 = (startIndex < 0) ? 0 : startIndex;
>+		final int length1 = (length < 0) ? Integer.MAX_VALUE : length;
>+
>+		final int partStartIndex = partDmc.getIndex();
>+		final int partLength = partDmc.getLength();
>+		if (partLength > PARTITION_LENGTH) {
>+			// create subpartitions
>+			int numChildren = computeNumberOfChildren(partLength);
>+			
>+			if (startIndex1 >= numChildren) {
>+				rm.setStatus(new Status(IStatus.ERROR, GdbPlugin.PLUGIN_ID, REQUEST_FAILED, "Invalid range for evaluating sub expressions.", null)); //$NON-NLS-1$
>+				rm.done();
>+				return;
>+			}
>+
>+			int numPart = Math.min(numChildren, length1);
>+			IndexedPartitionDMC[] children = new IndexedPartitionDMC[numPart];
>+			int index = partStartIndex;
>+			for (int i = 0; i < startIndex1; ++i)
>+				index += computePartitionLength(partLength, i);
>+			for (int i = 0; i < children.length; ++i) {
>+				int childPartLength = computePartitionLength(partLength, i + startIndex1);
>+				children[i] = createIndexedPartition(
>+					partDmc, 
>+					partDmc.getParentInfo(), 
>+					index, 
>+					childPartLength);
>+				index += childPartLength;
>+			}
>+			rm.setData(children);
>+			rm.done();
>+		}
>+		else {
>+			// this is the last partition level, create "real" children
>+			getRealSubExpressions(createExpression(partDmc.getParents()[0], partDmc.getParentInfo()), partDmc.getIndex() + startIndex, length, rm);
>+		}
>+	}
>+
>+	void getRealSubExpressions(
>+			final IExpressionDMContext exprCtx, 
>+			int startIndex,
>+			int length, 
>+			final DataRequestMonitor<IExpressionDMContext[]> rm) {
>+		
>+		ExprMetaGetChildren getChildren = (startIndex < 0 || length < 0) ? 
>+				new ExprMetaGetChildren(exprCtx) : new ExprMetaGetChildren(exprCtx, startIndex + length);
>+		final int startIndex1 = (startIndex < 0) ? 0 : startIndex;
>+		final int length1 = (length < 0) ? Integer.MAX_VALUE : length;
>+		fExpressionCache.execute(
>+				getChildren,				
>+				new DataRequestMonitor<ExprMetaGetChildrenInfo>(getExecutor(), rm) {
>+					@Override
>+					protected void handleSuccess() {
>+						ExpressionInfo[] childrenExpr = getData().getChildrenExpressions();
>+
>+						if (startIndex1 >= childrenExpr.length) {
>+							rm.setStatus(new Status(IStatus.ERROR, GdbPlugin.PLUGIN_ID, REQUEST_FAILED, "Invalid range for evaluating sub expressions.", null)); //$NON-NLS-1$
>+							rm.done();
>+							return;
>+						}
>+
>+						int numChildren = childrenExpr.length - startIndex1;
>+						numChildren = Math.min(length1, numChildren);
>+						IExpressionDMContext[] childrenArray = new IExpressionDMContext[numChildren];
>+						for (int i=0; i < numChildren; i++) {
>+							childrenArray[i] = createExpression(exprCtx.getParents()[0], childrenExpr[startIndex1 + i]);
>+						}
>+						rm.setData(childrenArray);
>+						rm.done();
>+					}
>+				});
>+	}
>+
>+	/**
>+	 * Returns the number of "real" children if it is less or equal to the partition size, 
>+	 * otherwise returns the number of partitions. 
>+	 */
>+	private int computeNumberOfChildren(int realNumberOfChildren) {
>+		int childNum = realNumberOfChildren;
>+		int extra = (childNum > PARTITION_LENGTH && childNum % PARTITION_LENGTH > 0) ? 1 : 0;
>+		while (childNum > PARTITION_LENGTH) {
>+			childNum /= PARTITION_LENGTH;
>+		}
>+		childNum += extra;
>+		return childNum;
>+	}
>+
>+	private int computePartitionLength(int realNumberOfChildren, int index) {
>+		int childNum = realNumberOfChildren;
>+		int depth = 0;
>+		int length = PARTITION_LENGTH;
>+		while (childNum > PARTITION_LENGTH) {
>+			childNum /= PARTITION_LENGTH;
>+			if (depth > 0)
>+				length *= PARTITION_LENGTH;
>+			++depth;
>+		}
>+		int diff = realNumberOfChildren - length*index;
>+		return ( diff > length ) ? length : diff ;
>+	}
>+
>+	private IndexedPartitionDMC createIndexedPartition(IDMContext ctx, ExpressionInfo info, int index, int length) {
>+	    IFrameDMContext frameDmc = DMContexts.getAncestorOfType(ctx, IFrameDMContext.class);
>+	    if (frameDmc != null) {
>+	        return new IndexedPartitionDMC(getSession().getId(), info, frameDmc, index, length);
>+	    } 
>+	    
>+	    IMIExecutionDMContext execCtx = DMContexts.getAncestorOfType(ctx, IMIExecutionDMContext.class);
>+	    if (execCtx != null) {
>+	    	// If we have a thread context but not a frame context, we give the user
>+	    	// the expression as per the top-most frame of the specified thread.
>+	    	// To do this, we create our own frame context.
>+	    	MIStack stackService = getServicesTracker().getService(MIStack.class);
>+	    	if (stackService != null) {
>+	    		frameDmc = stackService.createFrameDMContext(execCtx, 0);
>+	            return new IndexedPartitionDMC(getSession().getId(), info, frameDmc, index, length);
>+	    	}
>+        } 
>+
>+        return new IndexedPartitionDMC(getSession().getId(), info, ctx, index, length);
>+	}
>+
>+	private void getRealSubExpressionCount(IExpressionDMContext dmc, int numChildLimit, final DataRequestMonitor<Integer> rm) {
>+		if (dmc instanceof MIExpressionDMC) {
>+			fExpressionCache.execute(
>+					new ExprMetaGetChildCount(dmc, numChildLimit),				
>+					new DataRequestMonitor<ExprMetaGetChildCountInfo>(getExecutor(), rm) {
>+						@Override
>+						protected void handleSuccess() {
>+							rm.setData(getData().getChildNum());
>+							rm.done();
>+						}
>+					});
>+		} else if (dmc instanceof InvalidContextExpressionDMC) {
>+			rm.setStatus(new Status(IStatus.ERROR, GdbPlugin.PLUGIN_ID, INVALID_HANDLE, "Invalid context for evaluating expressions.", null)); //$NON-NLS-1$
>+			rm.done();
>+		} else {
>+			rm.setStatus(new Status(IStatus.ERROR, GdbPlugin.PLUGIN_ID, INTERNAL_ERROR, "Invalid expression context.", null)); //$NON-NLS-1$
>+			rm.done();
>+		}
>+	}
> }
>diff --git a/dsf-gdb/org.eclipse.cdt.dsf.gdb/src/org/eclipse/cdt/dsf/mi/service/MIVariableManager.java b/dsf-gdb/org.eclipse.cdt.dsf.gdb/src/org/eclipse/cdt/dsf/mi/service/MIVariableManager.java
>index 1636f91..2aa69e0 100644
>--- a/dsf-gdb/org.eclipse.cdt.dsf.gdb/src/org/eclipse/cdt/dsf/mi/service/MIVariableManager.java
>+++ b/dsf-gdb/org.eclipse.cdt.dsf.gdb/src/org/eclipse/cdt/dsf/mi/service/MIVariableManager.java
>@@ -31,6 +31,7 @@
> import org.eclipse.cdt.dsf.datamodel.DMContexts;
> import org.eclipse.cdt.dsf.debug.service.IExpressions;
> import org.eclipse.cdt.dsf.debug.service.IExpressions.IExpressionDMContext;
>+import org.eclipse.cdt.dsf.debug.service.IExpressions.IIndexedPartitionDMContext;
> import org.eclipse.cdt.dsf.debug.service.IExpressions2.ICastedExpressionDMContext;
> import org.eclipse.cdt.dsf.debug.service.IFormattedValues;
> import org.eclipse.cdt.dsf.debug.service.IFormattedValues.FormattedValueDMContext;
>@@ -1274,6 +1275,9 @@
> 	        		exprName = '(' + exprName + ')';
> 	        		castingIndex = ((ICastedExpressionDMContext)exprDmc).getCastInfo().getArrayStartIndex();
> 	        	}
>+	        	if (exprDmc instanceof IIndexedPartitionDMContext) {
>+	        		castingIndex = ((IIndexedPartitionDMContext)exprDmc).getIndex();
>+	        	}
> 	        	for (int i= 0; i < childrenOfArray.length; i++) {
> 	        		String fullExpr = exprName + "[" + i + "]";//$NON-NLS-1$//$NON-NLS-2$
> 	        		String relExpr = exprDmc.getRelativeExpression() + "[" + (castingIndex + i) + "]";//$NON-NLS-1$//$NON-NLS-2$
>diff --git a/dsf/org.eclipse.cdt.dsf.ui/src/org/eclipse/cdt/dsf/debug/internal/ui/viewmodel/DsfCastToTypeSupport.java b/dsf/org.eclipse.cdt.dsf.ui/src/org/eclipse/cdt/dsf/debug/internal/ui/viewmodel/DsfCastToTypeSupport.java
>index e625c17..458f182 100644
>--- a/dsf/org.eclipse.cdt.dsf.ui/src/org/eclipse/cdt/dsf/debug/internal/ui/viewmodel/DsfCastToTypeSupport.java
>+++ b/dsf/org.eclipse.cdt.dsf.ui/src/org/eclipse/cdt/dsf/debug/internal/ui/viewmodel/DsfCastToTypeSupport.java
>@@ -25,6 +25,7 @@
> import org.eclipse.cdt.dsf.debug.service.IExpressions.IExpressionChangedDMEvent;
> import org.eclipse.cdt.dsf.debug.service.IExpressions.IExpressionDMContext;
> import org.eclipse.cdt.dsf.debug.service.IExpressions.IExpressionDMData;
>+import org.eclipse.cdt.dsf.debug.service.IExpressions.IIndexedPartitionDMContext;
> import org.eclipse.cdt.dsf.debug.service.IExpressions2;
> import org.eclipse.cdt.dsf.debug.service.IExpressions2.CastInfo;
> import org.eclipse.cdt.dsf.debug.service.IExpressions2.ICastedExpressionDMContext;
>@@ -90,6 +91,9 @@
> 	    }
> 
> 		private boolean isValid() {
>+			if (exprDMC instanceof IIndexedPartitionDMContext)
>+				return false;
>+
> 	        TestExpressions2Query query = new TestExpressions2Query();
> 	        dmvmProvider.getSession().getExecutor().execute(query);
> 
>diff --git a/dsf/org.eclipse.cdt.dsf.ui/src/org/eclipse/cdt/dsf/debug/ui/viewmodel/variable/VariableVMNode.java b/dsf/org.eclipse.cdt.dsf.ui/src/org/eclipse/cdt/dsf/debug/ui/viewmodel/variable/VariableVMNode.java
>index f5bc8e0..6bc34c7 100644
>--- a/dsf/org.eclipse.cdt.dsf.ui/src/org/eclipse/cdt/dsf/debug/ui/viewmodel/variable/VariableVMNode.java
>+++ b/dsf/org.eclipse.cdt.dsf.ui/src/org/eclipse/cdt/dsf/debug/ui/viewmodel/variable/VariableVMNode.java
>@@ -37,6 +37,7 @@
> import org.eclipse.cdt.dsf.debug.service.IExpressions.IExpressionDMContext;
> import org.eclipse.cdt.dsf.debug.service.IExpressions.IExpressionDMData;
> import org.eclipse.cdt.dsf.debug.service.IExpressions.IExpressionDMLocation;
>+import org.eclipse.cdt.dsf.debug.service.IExpressions.IIndexedPartitionDMContext;
> import org.eclipse.cdt.dsf.debug.service.IExpressions2;
> import org.eclipse.cdt.dsf.debug.service.IExpressions3;
> import org.eclipse.cdt.dsf.debug.service.IExpressions3.IExpressionDMDataExtension;
>@@ -135,6 +136,11 @@
>      */    
>     public static final String PROP_VARIABLE_ADDRESS_CHANGED = ICachingVMProvider.PROP_IS_CHANGED_PREFIX + PROP_VARIABLE_ADDRESS;
> 
>+    /**
>+     * 'PROP_VARIABLE_BASIC_TYPE' property value for indexed partitions
>+     */
>+    private static final String INDEXED_PARTITION_TYPE = "indexed_partition_type"; //$NON-NLS-1$
>+    
>     private final SyncVariableDataAccess fSyncVariableDataAccess;
>     
>     /**
>@@ -330,6 +336,16 @@
>         };
>     };
>     
>+    public final static  LabelImage PARTITION_LABEL_IMAGE = new LabelImage(CDebugImages.DESC_OBJS_ARRAY_PARTITION) {
>+        { setPropertyNames(new String[] { PROP_VARIABLE_BASIC_TYPE }); }
>+        
>+        @Override
>+        public boolean isEnabled(IStatus status, java.util.Map<String,Object> properties) {
>+            String type = (String)properties.get(PROP_VARIABLE_BASIC_TYPE);
>+            return INDEXED_PARTITION_TYPE.equals(type);
>+        };
>+    };
>+    
>     protected IElementLabelProvider createLabelProvider() {
>     	
>     	//
>@@ -407,6 +423,7 @@
>                     new String[] { PROP_NAME }),
>                 POINTER_LABEL_IMAGE,
>                 AGGREGATE_LABEL_IMAGE, 
>+                PARTITION_LABEL_IMAGE,
>                 SIMPLE_LABEL_IMAGE,
>                 new StaleDataLabelForeground(),
>                 new VariableLabelFont(),
>@@ -422,6 +439,7 @@
>                     new String[] { PROP_ELEMENT_EXPRESSION }),
>                 POINTER_LABEL_IMAGE,
>                 AGGREGATE_LABEL_IMAGE, 
>+                PARTITION_LABEL_IMAGE,
>                 SIMPLE_LABEL_IMAGE,
>                 new StaleDataLabelForeground(),
>                 new VariableLabelFont(),
>@@ -770,10 +788,13 @@
>                                 // In case of an error fill in the expression text in the name column and expressions columns.
>                                 IExpressionDMContext dmc = findDmcInPath(update.getViewerInput(), update.getElementPath(), IExpressions.IExpressionDMContext.class);
>                                 if (dmc != null && dmc.getExpression() != null) {
>-                                    update.setProperty(PROP_NAME, dmc.getExpression());
>+                                	String displayName = getExpressionDisplayName(dmc);
>+                                    update.setProperty(PROP_NAME, displayName);
>                                     if (expression == null) {
>-                                        update.setProperty(PROP_ELEMENT_EXPRESSION, dmc.getExpression());
>+                                        update.setProperty(PROP_ELEMENT_EXPRESSION, displayName);
>                                     }
>+                                    if (dmc instanceof IIndexedPartitionDMContext)
>+                                    	update.setProperty(PROP_VARIABLE_BASIC_TYPE, INDEXED_PARTITION_TYPE);
>                                 }
>                                 update.setStatus(getStatus());
>                             }
>@@ -814,11 +835,23 @@
>     @ConfinedToDsfExecutor("getSession().getExecutor()")
>     protected void fillExpressionDataProperties(IPropertiesUpdate update, IExpressionDMData data) 
>     {
>-        update.setProperty(PROP_NAME, data.getName());
>+        IExpressionDMContext dmc = findDmcInPath(update.getViewerInput(), update.getElementPath(), IExpressions.IExpressionDMContext.class);
>+        String displayName = data.getName();
>+        if (dmc != null && dmc.getExpression() != null) {
>+        	displayName = getExpressionDisplayName(dmc);
>+            update.setProperty(PROP_NAME, displayName);
>+        }
>         update.setProperty(PROP_VARIABLE_TYPE_NAME, data.getTypeName());
>-        IExpressionDMData.BasicType type = data.getBasicType();
>-        if (type != null) {
>-            update.setProperty(PROP_VARIABLE_BASIC_TYPE, type.name());
>+        String typeValue = null;
>+        if (dmc instanceof IIndexedPartitionDMContext)
>+        	typeValue = INDEXED_PARTITION_TYPE;
>+        else {
>+            IExpressionDMData.BasicType type = data.getBasicType();
>+            if (type != null)
>+            	typeValue = type.name();
>+        }
>+        if (typeValue != null) {
>+            update.setProperty(PROP_VARIABLE_BASIC_TYPE, typeValue);
>         }
>         
>         //
>@@ -828,7 +861,7 @@
>         //
>         IExpression expression = (IExpression)DebugPlugin.getAdapter(update.getElement(), IExpression.class);
>         if (expression == null) {
>-            update.setProperty(AbstractExpressionVMNode.PROP_ELEMENT_EXPRESSION, data.getName());
>+            update.setProperty(AbstractExpressionVMNode.PROP_ELEMENT_EXPRESSION, displayName);
>         }
>     }
> 
>@@ -1344,4 +1377,19 @@
>             request.done();
>         }
>     }
>+
>+    /**
>+     * Returns the label for the element with the given context.
>+     */
>+    protected String getExpressionDisplayName(IExpressionDMContext dmc) {
>+    	String name = dmc.getExpression();
>+    	if (dmc instanceof IIndexedPartitionDMContext) {
>+    		IIndexedPartitionDMContext ipDmc = (IIndexedPartitionDMContext)dmc;
>+    		name = String.format(
>+    				"[%d...%d]",  //$NON-NLS-1$
>+    				ipDmc.getIndex(), 
>+    				ipDmc.getIndex() + ipDmc.getLength() - 1);
>+    	}
>+    	return name;
>+    }
> }
>diff --git a/dsf/org.eclipse.cdt.dsf/src/org/eclipse/cdt/dsf/debug/service/IExpressions.java b/dsf/org.eclipse.cdt.dsf/src/org/eclipse/cdt/dsf/debug/service/IExpressions.java
>index ba32cdc..fac33c4 100644
>--- a/dsf/org.eclipse.cdt.dsf/src/org/eclipse/cdt/dsf/debug/service/IExpressions.java
>+++ b/dsf/org.eclipse.cdt.dsf/src/org/eclipse/cdt/dsf/debug/service/IExpressions.java
>@@ -44,6 +44,30 @@
>     }
> 
>     /**
>+     * To avoid SWT performance issues large arrays are divided into partitions. 
>+     * This interface represents the context of such a partition.
>+     * 
>+     * @since 2.3
>+     */
>+    public interface IIndexedPartitionDMContext extends IExpressionDMContext {
>+
>+		/**
>+		 * Returns the expression string of the parent array.
>+		 */
>+		public String getParentExpression();
>+
>+		/**
>+		 * Returns the index of the partition's first element in the parent array. 
>+		 */
>+		public int getIndex();
>+		
>+		/**
>+		 * Returns the number of array's elements in the partition.
>+		 */
>+		public int getLength();
>+	}
>+
>+    /**
>      * The address and size of an expression.
>      */
>     public interface IExpressionDMAddress {

Flags:

nobody: iplog-

Actions: View | Diff

Attachments on bug 365541: 209289 | 209290 | 209291 | 209483 | 209487 | 209858 | 209859 | 209927 | 210087 | 210133 | 210145 | 210150