FCL/sftools/ana/dynaanaapps: comparison sysperfana/analyzetoolext/com.nokia.s60tools.analyzetool/src/com/nokia/s60tools/analyzetool/internal/ui/graph/MemOpDescriptor.java

equal deleted inserted replaced

-:844b047e260d
+:f65f740e69f9
 /*
-* Copyright (c) 2008-2009 Nokia Corporation and/or its subsidiary(-ies).
+* Copyright (c) 2008-2010 Nokia Corporation and/or its subsidiary(-ies).
 * All rights reserved.
 * This component and the accompanying materials are made available
 * under the terms of "Eclipse Public License v1.0"
 * which accompanies this distribution, and is available
 * at the URL "http://www.eclipse.org/legal/epl-v10.html".
 *
 */
 package com.nokia.s60tools.analyzetool.internal.ui.graph;
+import java.io.IOException;
 import java.util.AbstractList;
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Locale;
 import org.eclipse.core.resources.IMarker;
 import org.eclipse.core.resources.IProject;
 import org.eclipse.core.resources.IWorkspaceRoot;
 import org.eclipse.core.resources.ResourcesPlugin;
 import org.eclipse.core.runtime.CoreException;
+import org.eclipse.core.runtime.IStatus;
 import org.eclipse.core.runtime.Path;
 import org.eclipse.jface.viewers.CellEditor;
 import org.eclipse.jface.viewers.TextCellEditor;
 import org.eclipse.swt.SWT;
 import org.eclipse.swt.events.KeyListener;
 import com.nokia.carbide.cdt.builder.CarbideBuilderPlugin;
 import com.nokia.carbide.cdt.builder.project.ICarbideProjectInfo;
 import com.nokia.cdt.debug.cw.symbian.symbolreader.IFunction;
 import com.nokia.cdt.debug.cw.symbian.symbolreader.ISourceLocation;
 import com.nokia.cdt.debug.cw.symbian.symbolreader.ISymbolFile;
+import com.nokia.s60tools.analyzetool.Activator;
+import com.nokia.s60tools.analyzetool.engine.ICallstackManager;
 import com.nokia.s60tools.analyzetool.engine.statistic.AllocCallstack;
 import com.nokia.s60tools.analyzetool.engine.statistic.AllocInfo;
 import com.nokia.s60tools.analyzetool.engine.statistic.BaseInfo;
 import com.nokia.s60tools.analyzetool.engine.statistic.DllLoad;
 import com.nokia.s60tools.analyzetool.engine.statistic.SourceFile;
 @SuppressWarnings("restriction")
 class MemOpDescriptor implements IPropertySource {
 	BaseInfo memInfo;
 	/** process Id descriptor */
-	private static final String PID_ID = "pid";
+	private static final String PID_ID = "pid"; //$NON-NLS-1$
-	private final DotTextPropertyDescriptor PID_DESCRIPTOR = new DotTextPropertyDescriptor(PID_ID, "Process Id");
+	private final DotTextPropertyDescriptor PID_DESCRIPTOR = new DotTextPropertyDescriptor(PID_ID, Messages.MemOpDescriptor_1);
 	/** Memory operation type Allocation/Leak/Free descriptor */
-	private static final String TYPE_ID = "type";
+	private static final String TYPE_ID = "type"; //$NON-NLS-1$
-	private final DotTextPropertyDescriptor TYPE_DESCRIPTOR = new DotTextPropertyDescriptor(TYPE_ID, "Operation Type");
+	private final DotTextPropertyDescriptor TYPE_DESCRIPTOR = new DotTextPropertyDescriptor(TYPE_ID, Messages.MemOpDescriptor_3);
 	/** Time descriptor */
-	private static final String TIME_ID = "time";
+	private static final String TIME_ID = "time"; //$NON-NLS-1$
-	private final DotTextPropertyDescriptor TIME_DESCRIPTOR = new DotTextPropertyDescriptor(TIME_ID, "Time");
+	private final DotTextPropertyDescriptor TIME_DESCRIPTOR = new DotTextPropertyDescriptor(TIME_ID, Messages.MemOpDescriptor_5);
 	/** size descriptor */
-	private static final String SIZE_ID = "size";
+	private static final String SIZE_ID = "size"; //$NON-NLS-1$
-	private final DotTextPropertyDescriptor SIZE_DESCRIPTOR = new DotTextPropertyDescriptor(SIZE_ID, "Size");
+	private final DotTextPropertyDescriptor SIZE_DESCRIPTOR = new DotTextPropertyDescriptor(SIZE_ID, Messages.MemOpDescriptor_7);
 	/** memory address descriptor */
-	private static final String ADDR_ID = "address";
+	private static final String ADDR_ID = "address"; //$NON-NLS-1$
-	private final DotTextPropertyDescriptor ADDR_DESCRIPTOR = new DotTextPropertyDescriptor(ADDR_ID, "Address");
+	private final DotTextPropertyDescriptor ADDR_DESCRIPTOR = new DotTextPropertyDescriptor(ADDR_ID, Messages.MemOpDescriptor_9);
 	/** total memory size consumed by the process descriptor */
-	private static final String TSIZE_ID = "total";
+	private static final String TSIZE_ID = "total"; //$NON-NLS-1$
-	private final DotTextPropertyDescriptor TSIZE_DESCRIPTOR = new DotTextPropertyDescriptor(TSIZE_ID, "Total Size");
+	private final DotTextPropertyDescriptor TSIZE_DESCRIPTOR = new DotTextPropertyDescriptor(TSIZE_ID, Messages.MemOpDescriptor_11);
 	/** life time of an allocation descriptor. This applies only to non Leaked allocations  */
-	private static final String LIFETIME_ID = "lifetime";
+	private static final String LIFETIME_ID = "lifetime"; //$NON-NLS-1$
-	private final DotTextPropertyDescriptor LIFETIME_DESCRIPTOR = new DotTextPropertyDescriptor(LIFETIME_ID, "Life Time");
+	private final DotTextPropertyDescriptor LIFETIME_DESCRIPTOR = new DotTextPropertyDescriptor(LIFETIME_ID, Messages.MemOpDescriptor_13);
-	private static final String ATTRIBUTES_GROUP = "Attributes";
+	private static final String ATTRIBUTES_GROUP = "Attributes"; //$NON-NLS-1$
 	/** used for making absolute time values relative */
 	private long baseTime;
 	/** callstack item descriptor id*/
-	private static final String CALL_STACK_ID = "callstack";
+	private static final String CALL_STACK_ID = "callstack"; //$NON-NLS-1$
 	/** current project */
 	IProject iCurrentProject = null;
 	/** Symbol Reader */
 	SymReader iSymReader = null;
 	/** c++ files from the current project */
 	AbstractList<String> cppFileNames;
-	static final String LINE_SEPARATOR = " :: ";
+	static final String LINE_SEPARATOR = " :: "; //$NON-NLS-1$
+	private ICallstackManager callstackManager;
 	/**
 	 * Constructor
-	 * @param baseTime usually process start time, used for making absolute time values relative
+	 * @param newBaseTime usually process start time, used for making absolute time values relative
 	 * @param info the alloc or free info
 	 * @param project IProject to use for locating source file
 	 * @param symReader the SymReader to use for pinpointing
 	 * @param cppFiles
-	 */
+	 * @param callstackManager CallstackManager for reading callstacks from BaseInfo
-	public MemOpDescriptor(Long baseTime, BaseInfo info, IProject project, SymReader symReader, AbstractList<String> cppFiles) {
+	 */
+	public MemOpDescriptor(Long newBaseTime, BaseInfo info, IProject project, SymReader symReader, AbstractList<String> cppFiles, ICallstackManager callstackManager) {
 		memInfo = info;
 		iCurrentProject = project;
 		iSymReader = symReader;
 		cppFileNames = cppFiles;
-		this.baseTime = baseTime;
+		this.baseTime = newBaseTime;
+		this.callstackManager = callstackManager;
 	}
 	/* (non-Javadoc)
 	 * @see org.eclipse.ui.views.properties.IPropertySource#getEditableValue()
 	 */
 		if (memInfo instanceof AllocInfo && ((AllocInfo)memInfo).isFreed()) {
 			LIFETIME_DESCRIPTOR.setCategory(ATTRIBUTES_GROUP);
 			completeList.add(LIFETIME_DESCRIPTOR); //only non leaks
 		}
 		// add callstack descriptors
-		AbstractList<AllocCallstack> callstack = memInfo.getCallstack();
+		if (callstackManager != null && callstackManager.hasCallstack(memInfo)){
-		for (int i = 0; i < callstack.size(); i++) {
+			try {
+				List<AllocCallstack> callstack = callstackManager.readCallstack(memInfo);
-			final DotTextPropertyDescriptor propDesc = new DotTextPropertyDescriptor(i, CALL_STACK_ID);
+				if (callstack != null){
-			propDesc.setCategory("CallStack");
+					for (int i = 0; i < callstack.size(); i++) {
-			completeList.add(propDesc);
+						final DotTextPropertyDescriptor propDesc = new DotTextPropertyDescriptor(i, CALL_STACK_ID);
+						propDesc.setCategory("CallStack"); //$NON-NLS-1$
+						completeList.add(propDesc);
+					}
+				}
+			} catch (IOException e) {
+				//since callstacks aren't fatal and we can't handle it usefully here, log this to the error log
+				Activator.getDefault().log(IStatus.ERROR, Messages.MemOpDescriptor_18, e);
+			}
 		}
 		return completeList.toArray(new TextPropertyDescriptor[completeList.size()]);
 	}
 	/* (non-Javadoc)
 		//Note, this method must return String values for the CellEditor to work
 		if (PID_ID.equals(id)) {
 			return String.valueOf(memInfo.getProcessID());//process ids are usually decimal values
 		}
 		if (TYPE_ID.equals(id))
-			return memInfo instanceof AllocInfo ? ((AllocInfo)memInfo).isFreed() ? "Allocation" : "Leak" : "Free";
+			return memInfo instanceof AllocInfo ? ((AllocInfo)memInfo).isFreed() ? Messages.MemOpDescriptor_19 : Messages.MemOpDescriptor_20 : Messages.MemOpDescriptor_21;
 		if (TIME_ID.equals(id))
 			return GraphUtils.renderTime(memInfo.getTime() - baseTime);
 		if (SIZE_ID.equals(id))
-			return String.format("%,d B",memInfo.getSizeInt());
+			return String.format(Messages.MemOpDescriptor_22,memInfo.getSizeInt());
 		if (ADDR_ID.equals(id))
 			return Long.toString(memInfo.getMemoryAddress(),16);
 		if (TSIZE_ID.equals(id))
-			return String.format("%,d B",memInfo.getTotalMem());
+			return String.format(Messages.MemOpDescriptor_23,memInfo.getTotalMem());
 		if (LIFETIME_ID.equals(id)) {
 			if (memInfo instanceof AllocInfo && ((AllocInfo)memInfo).isFreed()) {
 				AllocInfo info = (AllocInfo) memInfo;
 				return GraphUtils.renderTime(info.getFreedBy().getTime() - info.getTime());
 			}
-			throw new IllegalStateException("Should not happen because we did not provide a lifetime descriptor for leak and free");
+			throw new IllegalStateException("Should not happen because we did not provide a lifetime descriptor for leak and free"); //$NON-NLS-1$
 		}
-		if (id instanceof Integer && memInfo.getCallstack() != null){
+		if (id instanceof Integer && callstackManager.hasCallstack(memInfo)){
 			int callstackId = (Integer)id;
-			if (callstackId < memInfo.getCallstack().size()){
+			try {
-				AllocCallstack callstackItem = memInfo.getCallstack().get(callstackId);
+				List<AllocCallstack> callstackList = callstackManager.readCallstack(memInfo);
-				DllLoad tempLoad = callstackItem.getDllLoad();
+				if (callstackId < callstackList.size()){
-				long addr = callstackItem.getMemoryAddress();
+					AllocCallstack callstackItem = callstackList.get(callstackId);
+					DllLoad tempLoad = callstackItem.getDllLoad();
-				String name = String.format("%1$08x",addr);
+					long addr = callstackItem.getMemoryAddress();
-				if (tempLoad != null && callstackItem.getMemoryAddress()!= tempLoad.getStartAddress()) {
-					SourceFile aSourcefile = pinpoint(callstackItem.getMemoryAddress(),
+					String name = String.format(Messages.MemOpDescriptor_25,addr);
-							callstackItem.getDllLoad());
+					if (tempLoad != null && callstackItem.getMemoryAddress()!= tempLoad.getStartAddress()) {
-					if (aSourcefile != null) { //callstack resolved to a file-function-line
+						SourceFile aSourcefile = pinpoint(callstackItem.getMemoryAddress(),
-						return name + LINE_SEPARATOR + callstackItem.getDllLoad().getName() + LINE_SEPARATOR + aSourcefile.getFileName()+ LINE_SEPARATOR + aSourcefile.getFunctionName() + LINE_SEPARATOR + aSourcefile.getLineNumber();
+								callstackItem.getDllLoad());
+						if (aSourcefile != null) { //callstack resolved to a file-function-line
+							return name + LINE_SEPARATOR + callstackItem.getDllLoad().getName() + LINE_SEPARATOR + aSourcefile.getFileName()+ LINE_SEPARATOR + aSourcefile.getFunctionName() + LINE_SEPARATOR + aSourcefile.getLineNumber();
+						}
 					}
+					return name + (tempLoad != null ? LINE_SEPARATOR + tempLoad.getName() : ""); //$NON-NLS-1$
 				}
-				return name + (tempLoad != null ? LINE_SEPARATOR + tempLoad.getName() : "");
+			} catch (IOException e) {
-				}
+				//since callstacks aren't fatal and we can't handle it usefully here, log this to the error log
-			}
+				Activator.getDefault().log(IStatus.ERROR, Messages.MemOpDescriptor_27, e);
+			}
+		}
 		return null;
 	}
 	/* (non-Javadoc)
 	 * @see org.eclipse.ui.views.properties.IPropertySource#isPropertySet(java.lang.Object)
 	private class DotPropKeyMouseListener implements KeyListener, MouseListener {
 		//key
 		public void keyPressed(org.eclipse.swt.events.KeyEvent e) {
+			//do nothing by design
 		}
 		public void keyReleased(org.eclipse.swt.events.KeyEvent keyEvent) {
 			//System.out.println("key released.");
 			if (keyEvent.character == '\r') {
 				}
 			}
 		}
 		public void mouseDown(MouseEvent e) {
+			//do nothing by design
 		}
 		public void mouseUp(MouseEvent e) {
+			//do nothing by design
 		}
 	}
 	/**
 			IFunction func = symbolFile.findFunctionByAddress(bigAddress);
 			ISourceLocation loc = symbolFile.findSourceLocation(bigAddress);
 			if (func != null && loc != null) {
 				String sourceFile = loc.getSourceFile();
 				if (sourceFile == null
-						|| sourceFile.equalsIgnoreCase(""))
+						|| sourceFile.equalsIgnoreCase("")) //$NON-NLS-1$
 					return null;
 				int lineNumber = loc.getLineNumber();
 				if (lineNumber == 0)
 					return null;
 				String name = func.getName();
 				if (name == null
-						|| name.equalsIgnoreCase(""))
+						|| name.equalsIgnoreCase("")) //$NON-NLS-1$
 					return null;
 				/*
 				 * if( onlyForProjectFiles &&
 				 * !isFilePartOfTheProject(loc.getSourceFile()) ) return null;
 				 */
 	private void openEditor(String cppFileName, String lineNumber) {
 		//check that used information is given
 		//we need to know file name and file line number
 		//that we could open the right line in editor
-		if (cppFileName == null || ("").equals(cppFileName)
+		if (cppFileName == null || ("").equals(cppFileName) //$NON-NLS-1$
-				|| lineNumber == null || ("").equals(lineNumber)) {
+				|| lineNumber == null || ("").equals(lineNumber)) { //$NON-NLS-1$
 			return;
 		}
 		try {
 			IFile file = null;
 				IWorkspaceRoot myWorkspaceRoot = ResourcesPlugin.getWorkspace()
 						.getRoot();
 				IProject[] projects = myWorkspaceRoot.getProjects();
 				for (int i = 0; i < projects.length; i++) {
 					file = ResourcesPlugin.getWorkspace().getRoot().getFile(
-							new Path(projects[i].getName() + "\\"
+							new Path(projects[i].getName() + "\\" //$NON-NLS-1$
 									+ usedFileName));
 					// file found => skip the rest of the projects
 					if (file != null && file.exists()) {
 						break;

changeset 6	f65f740e69f9
parent 1	1050670c6980
child 15	0367d2db2c06