From cb81af5c42590f4a86f632e2b22a0cad0a88389e Mon Sep 17 00:00:00 2001 From: Gregory John Casamento Date: Thu, 12 Feb 2009 03:02:57 +0000 Subject: [PATCH] * Framework/PCEditorManager.m: Switch to editor and line in the -debuggerDidHitBreakpoint: method. * Modules/Debuggers/ProjectCenter/PCDebugger.m: Minor cleanup. * Modules/Debuggers/ProjectCenter/PCDebuggerView.m: Changes to -[PCDebuggerView logString:newLine:] to detect state and change it on debugger window. * Modules/Debuggers/ProjectCenter/Resources/PCDebugger.gorm: Left justify the status field. git-svn-id: svn+ssh://svn.gna.org/svn/gnustep/apps/projectcenter/trunk@27850 72102866-910b-0410-8b05-ffd578937521 --- ChangeLog | 37 ++++++++++++------ Framework/PCEditorManager.m | 2 +- Modules/Debuggers/ProjectCenter/PCDebugger.m | 1 - .../Debuggers/ProjectCenter/PCDebuggerView.m | 21 ++++++++++ .../Resources/PCDebugger.gorm/data.info | Bin 184 -> 184 bytes .../Resources/PCDebugger.gorm/objects.gorm | Bin 3122 -> 3147 bytes 6 files changed, 48 insertions(+), 13 deletions(-) diff --git a/ChangeLog b/ChangeLog index 1763890..6290338 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,19 +1,34 @@ +2009-02-11 19:45-EST Gregory John Casamento + + * Framework/PCEditorManager.m: Switch to editor and line in the + -debuggerDidHitBreakpoint: method. + * Modules/Debuggers/ProjectCenter/PCDebugger.m: Minor cleanup. + * Modules/Debuggers/ProjectCenter/PCDebuggerView.m: Changes to + -[PCDebuggerView logString:newLine:] to detect state and change + it on debugger window. + * Modules/Debuggers/ProjectCenter/Resources/PCDebugger.gorm: + Left justify the status field. + 2009-02-11 Sergii Stoian + * PCMenuController.m: (subprojectNew): Use renamed [projectManager openNewSubprojectPanel] method. - * Framework/PCProjectManager.m: Change all alert panel names according to - operation performed. Edit messages text of alert panels. Cleanup. - (newSubproject): Rename to openNewSubprojectPanel which is more informative. - (createSubproject): Change method definition to createSubproject:(id)sender - and merge code with old createSubproject:(id)sender. - (controlTextDidChange:): Activate 'Create' button only if subproject with - enetered name doesn't exist in project. + * Framework/PCProjectManager.m: Change all alert panel names + according to operation performed. Edit messages text of alert + panels. Cleanup. + (newSubproject): Rename to openNewSubprojectPanel which is + more informative. (createSubproject): Change method definition + to createSubproject:(id)sender and merge code with old + createSubproject:(id)sender. (controlTextDidChange:): Activate + 'Create' button only if subproject with entered name doesn't + exist in project. * Framework/PCProjectBrowser.m: - (nameOfSelectedCategory): Fix bug which prevent from removing subprojects. - Now selected category is checked against superproject category list. When - subproject name selected, name of superproject's category is returned. As - side effect ProjectBrowser tries to load editor for selected subproject's + (nameOfSelectedCategory): Fix bug which prevent from removing + subprojects. Now selected category is checked against superproject + category list. When subproject name selected, name of superproject's + category is returned. As side effect ProjectBrowser tries to load + editor for selected subproject's name. Need to fix. 2009-02-10 Sergii Stoian diff --git a/Framework/PCEditorManager.m b/Framework/PCEditorManager.m index ca82f22..7c9b428 100644 --- a/Framework/PCEditorManager.m +++ b/Framework/PCEditorManager.m @@ -394,8 +394,8 @@ NSString *PCEditorDidResignActiveNotification = id editor = [self openEditorForFile: filePath editable: YES windowed: NO]; + [self orderFrontEditorForFile:filePath]; [editor scrollToLineNumber: [line intValue]]; - // NSLog(@"object = %@", [aNotif object]); } @end diff --git a/Modules/Debuggers/ProjectCenter/PCDebugger.m b/Modules/Debuggers/ProjectCenter/PCDebugger.m index 80b41aa..c85bca2 100644 --- a/Modules/Debuggers/ProjectCenter/PCDebugger.m +++ b/Modules/Debuggers/ProjectCenter/PCDebugger.m @@ -141,7 +141,6 @@ static NSImage *downImage = nil; [debuggerWindow setToolbar: toolbar]; RELEASE(toolbar); - [toolbar setUsesStandardBackgroundColor: YES]; [debuggerView setFont: [NSFont userFixedPitchFontOfSize: 0]]; [debuggerWindow setFrameAutosaveName: @"PCDebuggerWindow"]; [self setStatus: @"Idle."]; diff --git a/Modules/Debuggers/ProjectCenter/PCDebuggerView.m b/Modules/Debuggers/ProjectCenter/PCDebuggerView.m index d01e02b..d3ce289 100644 --- a/Modules/Debuggers/ProjectCenter/PCDebuggerView.m +++ b/Modules/Debuggers/ProjectCenter/PCDebuggerView.m @@ -106,6 +106,27 @@ subProcessId = [process intValue]; } + // Check certain status messages from GDB and set the state correctly. + range = [str rangeOfString: @"Starting program:"]; + if (range.location != NSNotFound) + { + [debugger setStatus: @"Running..."]; + } + + // Check certain status messages from GDB and set the state correctly. + range = [str rangeOfString: @"Program recieved signal:"]; + if (range.location != NSNotFound) + { + [debugger setStatus: @"Stopped"]; + } + + // Check certain status messages from GDB and set the state correctly. + range = [str rangeOfString: @"Terminated"]; + if (range.location != NSNotFound) + { + [debugger setStatus: @"Terminated"]; + } + // FIXME: Filter this error, until we find a better way to deal with it. range = [str rangeOfString: @"[tcsetpgrp failed in terminal_inferior:"]; if (range.location != NSNotFound) diff --git a/Modules/Debuggers/ProjectCenter/Resources/PCDebugger.gorm/data.info b/Modules/Debuggers/ProjectCenter/Resources/PCDebugger.gorm/data.info index 336424908965f092fd6d62dd667d63cc1130a3c4..da98e4327f70ff8ec64d5cb98405097d84145e8a 100644 GIT binary patch delta 11 ScmdnNxPx&*8YAb#^gsX?%mcgt delta 11 ScmdnNxPx&*8Y9QV^gsX?$OF3o diff --git a/Modules/Debuggers/ProjectCenter/Resources/PCDebugger.gorm/objects.gorm b/Modules/Debuggers/ProjectCenter/Resources/PCDebugger.gorm/objects.gorm index 8cb7030298bb4e9df92f216b80e530398dd72e2b..110fa6c3a2e172301b0e52a6c14a1866aa0c530d 100644 GIT binary patch delta 211 zcmdlaaav-62BYCd&0VY<3`~q_3=9lBn_shLGBbYI9LibC$YfwY`2we;;{*eTjj|hT zHdszDa0D_eHrPxsZ~`){HdstBa5j`<5b+E4%S`ewEy+nOan8@nOHD4xF9MpzzyMTZ zZ{RXnl-rQgb)(z{OCZl}vOl*pr#qNuW#BQnm|KI>6UejQU}@krc?q{7r#F~qW#BXU o9Jjo*FVH4akd=NwhBb)c50tu}NZr2BX17&0VYn_shLGBcjo9LibCsIb9of`R=;*$p-uEGHN^ z02vk=Y$g~u0vT2tEG8H@P3GoSU^X*wo~*}hsOz#(ZiD3pGXvL+@*tBqRDlGmD2T-D<;09J>W#B%!hg*Zw1IRPmU}@kvc`vsjrx%!KW#B#eIk&vD56}`*kZry| chBb)c2bQ-s@Sm*6BgYf4Q5s|mBhV#00Q)07$^ZZW