Use our new error system

* Source/GDBpConnection.m:
(processData:): Call -[errorEncountered]

Create an error system that follows MVC

* Source/DebuggerController.m:
(init): Register ourselves as an observer for the error notif type
(handleConnectionError:): New method to handle the error notifs
* Source/GDBpConnection.h: Forward-declare kErrorOccurred
* Source/GDBpConnection.m:
(errorEncountered:): Post the notification

Remove all of the old stack system

* Source/DebuggerController.h: Remove stackController2 and stack ivars
* Source/DebuggerController.m:
(validateUserInterfaceItem:): Update to use stackController.stack for -[count]
(resetDisplays): Remove the call to stackController2
(setStack): Removed
(stepIn:): Call -[updateStackViewer]
(stepOut:): ditto
(stepOver:): ditto
(updateSourceViewer): Use our new stackController
(updateStackViewer): New method
* Source/GDBpConnection.m:
(updateStackTraceAndRegisters): Remove the block of code that used to update the stack

Update the bindings for the stack viewer to use our new StackController system

* English.lproj/Debugger.xib: Redo the bindings
* Source/DebuggerController.h: Add stackArrayController ivar
* Source/DebuggerController.m:
(awakeFromNib): Set the sort descriptors
(stepIn:): Rearrange the stackArrayController
(stepOut:): ditto
(stepOver): ditto

The StackFrame index will always be 0 in -[GDBpConnection createStackFrame], so we need to advance it in -[StackController pop] and -[push:]

* Source/GDBpConnection.m:
(createStackFrame): Don't bother getting the "level" property, it will always be 0
* Source/StackController.m:
(pop): Decrease all of the frames' counter
(push:): Increase all of the frames' counter

We need to pop the stack twice when stepOut is called

* Source/DebuggerController.m:
(stepOut): Pop the stack twice

Start using our new StackFrame and StackController classes as struts

* Source/DebuggerController.h: Add the stackController ivar
* Source/DebuggerController.m:
(init): Initialize the stackController
(dealloc): Release stackController
(stepIn:): Pop and push the stackController appropriately
(stepOut:): ditto
(stepOver:): ditto
* Source/GDBpConnection.h: -[stepIn] -[stepOut] and -[stepOver] now all return StackFrame objects
* Source/GDBpConnection.m:
(stepIn): Generate and return the frame
(stepOut): ditto
(stepOver): ditto
(createStackFrame): New private method
* Source/StackController.m+h:
(peek): New method
* Source/StackFrame.m+h:
(isShiftedFrame:): New method
(description): Implemented

Refactor the DebuggerController's "stackController" ivar to be named "stackController2"

Add the StackController class

* Source/StackController.m+h: New files
* MacGDBp.xcodeproj: Project updates for new files

Adding the StackFrame class

* Source/StackFrame.m+h: New files
* MacGDBp.xcodeproj: Project updates

Reorganize the project files to separate connection from debugger classes

* MacGDBp.xcodeproj

Update version number

* Info.plist

Have the debugger and breakpoints windows remember their size and location

* English.lproj/Breakpoints.xib: Set autosave name
* English.lproj/Debugger.xib: ditto
* Source/DebuggerController.m:
(awakeFromNib): Do not call [window center] anymore

Saved breakpoints (or any breakpoint) that had a file that does not exist would crash MacGDBp

* Source/BSSourceView.m:
(setFile:): If the file does not exist, simply display an empty string
(scrollToLine:): If the file does not exist, return

Update the version to 1.1.1

* Info.plist

Allow multiple selection and removal of breakpoints

* English.lproj/Breakpoints.xib: Allow multiple selection
* Source/BreakpointController.m:
(removeBreakpoint:): Remove the selection's entirety, instead of the first element

Fix a bug where breakpoints would be recreated exponentially after their removal

* Source/BreakpointManager.m:
(removeBreakpointAt:inFile:): Need to call -[removeObject:] instead of -[addObject:]

Add the bug numbers to the change log

* CHANGES

Preferences should be set in +load rather than +initialize, as we can’t rely on AppDelegate receiving a message before any other objects (In this case, DebuggerController was being created first, leading to sockets without the correct default port number).

* Source/AppDelegate.m:
(initialize): Change to load

Changing the version to be 1.1

* Info.plist

Keep breakpoints in the preferences so they are restored at launch

* Source/BreakpointManager.m+h: Add a savedBreakpoints ivar to hold the dictionarys
(addBreakpoint:): Add the breakpoint to the NSUserDefaults
(removeBreakpointAt:inFile:): Remove the breakpoint from NSUserDefaults

Add a CHANGES file to manage the change log

* CHANGES

Fixing a memory leak

* Source/DebuggerController.m:
(gutterClickedAtLine:forFile:): The breakpoint (bp) sent to BreakpointManager is over-retained

Check and see if stderr has any data on it when syntax highlighting. If so, only show plain text.

* Source/BSSourceView.m:
(initWithFrame): Register to be an observer of the file handle reader notification
(setFile:): Add another pipe for stderr, and read in background from it
(errorHighlightingFile:): New method to handle the notification

Code formatting

Renaming NSXMLElementAdditions(NSXMLElementAdditions) to be NSXMLElementAdditions(GDBpAdditions)

Renaming BreakpointWindowController to BreakpointController

Renaming DebuggerConnection to GDBpConnection

Renaming DebuggerWindowController to DebuggerController

Add IBOutlets for both DebuggerWindowController and BreakpointWindowController

* Source/AppDelegate.h: Add the IBOutlets
* English.lproj/MainMenu.xib: Hook up the IBOs

-[SocketWrapper(Private) error:] would send message to a non-existent selector

* Source/SocketWrapper.m:
(error:): [errorEncountered:] has been replaced with [setError:]

The subviews of BSSourceView were over-retained

* Source/BSSourceView.m:
(dealloc): Call [removeFromSuperview] on the subviews

Update the version to 1.1 Beta 1

Fix build errors. Properties cannot be synthesized if they are defined in a category

* Source/SocketWrapper.m

Merge branch 'memory-fixes'

Validate user interface items for the Debugger menu

* Source/DebuggerWindowController.m:
(init): Set self to be [self window]'s delegate
(validateUserInterfaceItem:): New method

Fixing a bug in -[SocketWrapper receive] that could miss up to the last 8 bytes of a response

* Source/SocketWrapper.m:
(receive): A multi-packet response could be incomplete by a difference of 8 bytes because we didn't factor in packetLength when counting

Add a call to NSAssert() to stop us if we receive incomplete packets from SocketWrapper

* Source/SocketWrapper.m:
(receive): Assert that the packet is a complete XML document

A few miscellaneous memory management fixes

* Source/BSSourceView.m:
(setFile:): Autorelease the NSTask and release the NSAttributedString
* Source/BSSplitView.m:
(drawDividerInRect:): Release the NSGradient after drawing
* Source/Breakpoint.m:
(initWithDictionary:): Retain the file object
* Source/BreakpointManager.m:
(hasBreakpointAtLine:inFile:): Autorelease the object we send to [containsObject:]

The -connect: method would leak memory when called more than once

* Source/SocketWrapper.m: Add a private prop for the hostname
(remoteHost): Removed through use of a prop
(connect:): Change from alloc'ing an NSString to creating an autorelease'd one

A crash would occur in the event that -setFile: was called with the same path string object as is currently own

* Source/BSSourceView.m:
(setFile:)

Sending messages to nil is always safe (a no-op)

* Source/BSSourceView.m:
(dealloc): Don't need to check and see if file is nil before [release]

Merge branch 'menu-validation'

Conflicts:

English.lproj/MainMenu.xib: Took HEAD XIB and copied branch XIB's Debugger menu manually

Validate user interface items for the Debugger menu

* Source/DebuggerWindowController.m:
(init): Set self to be [self window]'s delegate
(validateUserInterfaceItem:): New method

Fixing a bug in -[SocketWrapper receive] that could miss up to the last 8 bytes of a response

* Source/SocketWrapper.m:
(receive): A multi-packet response could be incomplete by a difference of 8 bytes because we didn't factor in packetLength when counting

Add a call to NSAssert() to stop us if we receive incomplete packets from SocketWrapper

* Source/SocketWrapper.m:
(receive): Assert that the packet is a complete XML document

Adding the "Debugger" menu to the NIB file and hooking up the actions for them

* English.lproj/MainMenu.xib

Tracking the Italian versin of Sparkle

* Sparkle.framework/Versions/A/Resources/it.lproj/: Add

Add the Sparkle keys to Info.plist

* Info.plist

Hook up the "Check for updates..." menu item

* English.lproj/MainMenu.xib

Updating Sparkle to use a version that uses CFBundleShortVersionString for comparision instead of CFBundleVersion

* Sparkle.framework/

Remove the old version checking system that we wrote ourselves

* Source/AppDelegate.m+h: Removed updater outlets
(applicationDidFinishLaunching:): Removed
(versionCheck:): Removed
(openUpdateInformation:): Removed
* English.lproj/MainMenu.xib: Removed the update window

Add Sparkle framework and hook it up (except for the appcast URL)

* English.lproj/MainMenu.xib: Instantiate the SUUpdater
* MacGDBp.xcodeproj: Add a build phase and link against Sparkle.framework
* Sparkle.framewokr/: New files

Instantiate the Debugger and Breakpoint controllers in the NIB rather than in code

* English.lproj/MainMenu.xib: Instantiate DebuggerWindowController and BreakpointWindowController
* Source/AppDelegate.m:
(applicationDidFinishLaunching:): Remove the two object init lines

Use prefernce values to set the port and IDE key, rather than hard-coded values

* Source/DebuggerWindowController.m+h:
(initWithPort:session:): Renamed to -[init] and use preference values to set the port and IDE key
* Source/AppDelegate.m:
(applicationDidFinishLaunching:): Simply call init on DebuggerWindowController

Add a warning that says to close and reopen MacGDBp after changing pref values

* English.lproj/Preferences.xib

Bind the preference inputs to the NSUserDefaultsController

* English.lproj/Preferences.xib

Setting the default values for NSUserDefaults

* Source/AppDelegate.m:
(initialize): Register the default preferences dict

Laying out the user interface for the preferences window

* English.lproj/Preferences.xib

Adding an empty preferences window and controller. Hooking up the Preferences menu item.

* English.lproj/MainMenu.xib: Hooking up the Preferences menu item
* English.lproj/Preferences.xib: New file
* Source/PreferencesController.m+h: New files
* Source/AppDelegate.m+h: Adding an ivar for the PreferencesController
(showPreferences:): New method
* MacGDBp.xcodeproj: Project updates for new files

Merge branch 'bug-125'

Conflicts:

Source/BSSourceView.m

Instead of using the "stopping" status, just use "Stopped" for both itself and "stopping"

* Source/DebuggerConnection.m:
(refreshStatus)

Close the connection when we receive the "stopping" status. Fixes bug://report/125

* Source/DebuggerConnection.m:
(refreshStatus): The "stopped" and "stopping" statuses should have the same effect now

Make the markers prop in BSLineNumberView be retain instead of assign

* Source/BSLineNumberView.h

Balancing with a [release] call

* Source/BSLineNumberView.m:
(drawRect:)

Merge git://github.com/ciaran/macgdbp

We need to balanace with a [release] call

* Source/BreakpointWindowController.m:
(gutterClickedAtLine:forFile:)

Syntax highlight source files using the PHP executable

Tell the breakpoints window to open at launch, though in back of the debugger

* Source/BreakpointWindowController.m:
(init): [orderBack:] the window

Added a [release] call

* Source/AppDelegate.m:
(versionCheck:): Need to release the NSXMLDocument because we alloc'd it

Version 1.0.1

* Info.plist

Return the NSXMLDocument in [DebuggerConnection processData:] as autorelease

* Source/DebuggerConnection.m:
(processData:): [autorelease] the retval

Instead of leaking an alloc'd NSString, use an autorelease'd initializer

* Source/SocketWrapper.m:
(receive:): When building up the full NSString, use the non-alloc-izing initializer

The hostname was using an autorelease'd initializer, so switch to an alloc'd one

* Source/SocketWrapper.m:
(connect:): Use [NSString initWithUTF8String:] instead of [NSString stringWithUTF8String:]
(dealloc): Add the corresponding release call

Fix a few exceptions/warnings related to going no-gc

* Source/AppDelegate.m
(versionCheck:): Add an NSAutoreleasePool
* Source/BreakpointManager.h: Specify an assignment type for the connection ivar
* Source/SocketWrapper.m:
(connect:): Add an NSAutoreleasePool

Merge branch 'master' into no-gc

Center the version check window (when it pops up) and the main window

* Source/AppDelegate.m:
(versionCheck:)
* Source/DebuggerWindowController.m:
(awakeFromNib)

Have the help menu go to our website instead of saying no help avaliable

* Source/AppDelegate.m+h:
(openHelpPage:): New method
* English.lproj/MainMenu.xib: Change the IBA for the help action

Adding a *real* version checking system

* English.lproj/MainMenu.xib: Adding the version update window
* Source/AppDelegate.h: Adding IBOs for the updater window and the update information string
* Source/AppDelegate.m:
(init): Run the version check method
(versionCheck:): New method
(openUpdateInformation:): New method

This is going to be v1.0

* Info.plist

After we establish a connection, we need to resend any offline breakpoints

* Source/DebuggerConnection.m:
(socketDidAccept:)

Protect against array out of bounds exceptions when removing breakpoints

* Source/BreakpointWindowController.m:
(removeBreakpoint:): If there are no selected objects in the NS-AC, don't try to remove breakpoints

Change the font size of the breakpoint NS-TV to be size 11

* English.lproj/Breakpoints.xib

The breakpoint manager now works -- breakpoints can be set from the main debugger or the manager

* English.lproj/Breakpoints.xib: Set the NSTableView's delegate to be File's Owner and disconnect the content IBO to the NS-AC
* Source/BreakpointManager.m:
(updateDisplaysForfile:): New private method
(addBreakpoint:): Call [updateDisplaysForfile:]
(removeBreakpointAt:inFile:): ditto
* BreakpointWindowController.m+h: Add props and synth them for sourceView and arrayController ivars
(removeBreakpoint:): Fill out method stub
(tableViewSelectionDidChange:): Implement delegate method
* DebuggerWindowController.m+h: Add prop/synth for sourceViewer ivar

Fixing a bug where if the register depth was too deep, more properties would not be fetched/inserted correctly

* Source/AppDelegate.m+h: Add and synthesize props for debugger and breakpoint ivars
* Source/DebuggerConnection.m+h:
(getProperty:forNode:): Renamed [getProperty:] and now return an NSArray rather than attaching to a node
* Source/DebuggerWindowController.m+h:
(addChildren:toNode:): Removed
(outlineViewItemDidExpand:): Don't have to do extra fetch work here because it will now be done in the model
* Source/NSXMLElementAdditions.m+h:
(subnodes): New method that does the same thing as [children] but intelligently fetches more depth
* English.lproj/Debugger.xib: Tell the regiser NSTreeController to use [subnodes] instead of [children]

Another instance where [BSLineNumberView setMarkers:] takes a set and not an NSArray

* Source/BreakpointWindowController.m:
(gutterClickedAtLine:forFile:)

Trying to use mutableChildNodes and appending mock-NSTreeNode objects

Index path by appending IP with a counter

Index path by appending 0 to the IP

[BSLineNumberView setMarkers:] takes an NSSet and not an NSArray

* Source/DebuggerWindowController.m:
(gutterClickedAtLine:forFile:): Create an NSSet from the NSArray

Set the "Automatically Rearranges Content" and "Prepares Content" flags on the Breakpoints.xib NSArrayController

* English.lproj/Breakpoints.xib

In DebuggerConnection, we need to protect against setting breakpoints with no active connection

* Source/DebuggerConnection.m:
(addBreakpoint:)
(removeBreakpoint:)

Set the BSSourceView's delegate in Breakpoints.xib

* English.lproj/Breakpoints.xib

Have the BreakpointManager tell the DebuggerConnection about breakpoints, rather than controllers

* Source/BreakpointManager.m:
(addBreakpoint:)
(removeBreakpointAt:inFile:)
* Source/DebuggerWindowController.m:
(gutterClickedAtLine:forFile:)

Auto-hide the scrollbars in BSSourceView

* Source/BSSourceView.m:
(setupViews)

Automatically hide scrollbars on the NSTableView of breakpoints

* English.lproj/Breakpoints.xib

Set the BreakpointManager's connection to the DebuggerConnection instance

* Source/DebuggerConnection.m:
(initWithWindowController:port:session): Set the connection to be self

Add a prop for DebuggerConnection and change the breakpoints ivar to a prop

* Source/BreakpointManager.m+h

Hooking up the BSSourceView outlet and implement delegates

* English.lproj/Breakpoints.xib: Hook up outlet
* Source/BreakpointWindowController.h: Add the ivar
* Source/BreakpointWindowController.m:
(addBreakpoint:): Show the NSOpenPanel to select a file
(gutterClickedAtLine:forFile:): New method

Hooking up a NSArrayController to manage the breakpoints

* English.lproj/Breakpoints.xib
* Source/BreakpointWindowController.h: Add an ivar for the NSAC