fix log/sym folder creation failure, clean code

xwlan · Sep 22, 2015 · f416b81 · f416b81
1 parent 97cd909
commit f416b81
Show file tree

Hide file tree

Showing 8 changed files with 10 additions and 33 deletions.
diff --git a/apspf/aps.c b/apspf/aps.c
@@ -221,8 +221,10 @@ ApsInitialize(
 
 	StringCchPrintf(ApsLogPath, MAX_PATH, L"%s\\log", Path);
 	Status = CreateDirectory(ApsLogPath, NULL);
-	if (Status != APS_STATUS_OK && Status != ERROR_ALREADY_EXISTS) {
-		return Status;
+	if (!Status) { 
+		if (GetLastError() != ERROR_ALREADY_EXISTS) {
+			return APS_STATUS_ERROR;
+		}
 	}
 
 	//
@@ -231,8 +233,10 @@ ApsInitialize(
 
 	StringCchPrintf(ApsLocalSymPath, MAX_PATH, L"%s\\sym", Path);
 	Status = CreateDirectory(ApsLocalSymPath, NULL);
-	if (Status != APS_STATUS_OK && Status != ERROR_ALREADY_EXISTS) {
-		return Status;
+	if (!Status) { 
+		if (GetLastError() != ERROR_ALREADY_EXISTS) {
+			return APS_STATUS_ERROR;
+		}
 	}
 
 	Status = ApsInitializeLog(APS_FLAG_LOGGING);

diff --git a/dprofiler/cpuflame.c b/dprofiler/cpuflame.c
@@ -236,8 +236,6 @@ CpuFlameInsertData(
     PBTR_STACK_RECORD Record;
     ULONG Count;
     PCALL_GRAPH Graph;
-	PCPU_THREAD_TABLE ThreadTable;
-	PCPU_THREAD Thread;
 
     Object = (PDIALOG_OBJECT)SdkGetObject(hWnd);
     Context = (PCPU_FORM_CONTEXT)Object->Context;

diff --git a/dprofiler/cpuhistory.c b/dprofiler/cpuhistory.c
@@ -2304,7 +2304,7 @@ CpuHistoryInsertBackTrace(
 		lvi.iSubItem = 2;
 		lvi.mask = LVIF_TEXT;
 
-		if (Text->LineId != -1) {
+		if (Text != NULL && Text->LineId != -1) {
 			ASSERT(Line != NULL);
 			LineEntry = Line + Text->LineId;
 			StringCchPrintf(Buffer, MAX_PATH, L"%S:%u", LineEntry->File, LineEntry->Line);

diff --git a/dprofiler/cputhread.c b/dprofiler/cputhread.c
@@ -728,24 +728,6 @@ CpuThreadGetTable(
     return (PCPU_THREAD_TABLE)Context->Context;
 }
 
-PCPU_THREAD
-CpuThreadGetMostBusyThread(
-    __in HWND hWnd
-    )
-{
-    PDIALOG_OBJECT Object;
-    PCPU_FORM_CONTEXT Context;
-    PCPU_THREAD_TABLE Table;
-
-    Object = (PDIALOG_OBJECT)SdkGetObject(hWnd);
-    Context = (PCPU_FORM_CONTEXT)Object->Context;
-    Table = (PCPU_THREAD_TABLE)Context->Context;
-
-    ASSERT(Table != NULL);
-
-
-}
-
 VOID
 CpuThreadInsertThreads(
 	__in HWND hWnd,

diff --git a/dprofiler/cputree.c b/dprofiler/cputree.c
@@ -534,8 +534,6 @@ CpuTreeInsertData(
     PBTR_STACK_RECORD Record;
     ULONG Count;
     PCALL_GRAPH Graph;
-    PCPU_THREAD Thread;
-    PCPU_THREAD_TABLE ThreadTable;
 
     Object = (PDIALOG_OBJECT)SdkGetObject(hWnd);
     Context = (PCPU_FORM_CONTEXT)Object->Context;

diff --git a/dprofiler/find.c b/dprofiler/find.c
@@ -12,8 +12,6 @@ FindIsComplete(
 	__in ULONG Current
 	)
 {
-	ULONG Count;
-
 	if (Context->FindForward) {
 		return TRUE;
 	}

diff --git a/dprofiler/fullstack.c b/dprofiler/fullstack.c
@@ -206,7 +206,6 @@ FullStackInsertBackTrace(
     PBTR_TEXT_TABLE Table;
     PPF_REPORT_HEAD Report;
     PBTR_TEXT_ENTRY Text;
-    PBTR_DLL_ENTRY DllEntry;
     ULONG Length;
     PWCHAR Symbol;
     ULONG i;
@@ -507,7 +506,6 @@ FullStackOnTreeListSort(
 	PDIALOG_OBJECT Object;
     PFULLSTACK_CONTEXT Context;
 	PTREELIST_OBJECT TreeList;
-	ULONG Column;
 	TVSORTCB Tsc = {0};
     CPU_SORT_CONTEXT SortContext;
 	TVITEM tvi = {0};
@@ -518,7 +516,7 @@ FullStackOnTreeListSort(
     ASSERT(TreeList != NULL);
 
     SortContext.Column = wp;
-    SortContext.Context = NULL;
+    SortContext.Context = (LPARAM)NULL;
     SortContext.Order = TreeList->SortOrder;
 
 	Tsc.hParent = TVI_ROOT;

diff --git a/dprofiler/rebar.c b/dprofiler/rebar.c
@@ -267,7 +267,6 @@ RebarInsertBands(
 	HWND hWndCombo;
 	HWND hWndBar;
 	HWND hWndRebar;
-	HWND hWndEdit;
 	PREBAR_BAND Band;
 	HIMAGELIST Normal;
 	HIMAGELIST Grayed;