Adding a CONTRIBUTORS file

Word wrap CHANGES

Show the variables in all contexts instead of just Locals. Fixes #147.

Patch partial from Nicola Ferruzzi.

* Source/StackFrame.m+h: Change "contexts" to "variables"
(initWithIndex:withFilename:withSource:atLine:inFunction:withContexts:): Renamed to be ...withVariables:
* Source/GDBpConnection.m:
(createStackFrame): Collect all the contexts into an array and send that to StackFrame
* English.lproj/Debugger.xib: Rebind to simply "variables"

When the debugger connects, automatically step in to the first frame to make the debugger look active

* Source/DebuggerController.m+h:
(startDebugger): New method
* Source/GDBpConnection.m:
(socketDidAccept:): Call the -[startDebugger] method

Updating the changelog for 1.2.0

* CHANGES

Happy new year! Bump copyright.

Highlighting of the currently debugged file will now get the source from Xdebug rather than reading local files

* Source/BSSourceView.m+h:
(setString:asFile:): New method that will highlight strings rather than files
(scrollToLine:): Change the sanity check to be based off of text in the textStorage, rather than if a file has been set
* Source/DebuggerController.m:
(updateSourceViewer): Use the new setString:asFile: instead of just setFile:

Merge branch 'refactoring'

Conflicts:
English.lproj/Debugger.xib

Fixed a bug where stepIn would remove the current stack frame if the file was the same but a function call was made

* Source/StackFrame.m:
(isShiftedFrame:): A frame is shifted if it is in the same file AND it is still in the same function

Renaming registers to variables

We need to keep variables expanded across changing stack positions

* Source/DebuggerController.m:
(setRegister:): Removed
(tableViewSelectionDidChange:): Call [expandRegisters]
(expandVariables): New private method
* Source/DebuggerController.h: Header updates

Fix a couple display issues and a crash

* Source/DebuggerController.m:
(resetDisplays): Clear all the items in the stackController.stack
(updateSourceViewer): If the selection is empty, do not try to clear it. Also, tell the textView to repaint.

Get the source code from the debugger engine in [GDBpConnection(Private) createStackFrame]

* Source/GDBpConnection.m:
(createStackFrame): Use the source GDBp command to get the source code for the stack frame

Remove -[DebuggerController setStatus:]

* Source/DebuggerController.h: Header updates
* Source/DebuggerController.m:
(awakeFromNib): Set the window's title and remove the call to [setStatus:]
(setStatus:): Removed
(setError:): Remove call to [setStatus:]
* Source/GDBpConnection.m:
(initWithPort:session:): Set the status to "Connecting"

Use automatic validation of UI items for the toolbar buttons, rather than manually enabling/disabling them

* Source/DebuggerController.h: Remove the outlets for the toolbar items
* Source/DebuggerController.m:
(validateUserInterfaceItem:): Validate the reconnect button
(setStatus:): Remove the block of code that sets button states
(updateStackViewer): ditto
* English.lproj/Debugger.xib: Remove the outlets and set all the toolbar items to autovalidate

Call -[resetDisplays] when we reconnect

* Source/DebuggerController.m:
(reconnect:): Call resetDisplays

Use bindings to set the register controller's contentArray

* English.lproj/Debugger.xib

Remove all references of the window controller from GDBpConnection

* Source/GDBpConnection.h: Removed the windowController ivar and prop
* Source/GDBpConnection.m:
(initWithWindowController:port:session): Renamed to -[initWithPort:session:]
(dealloc): No more windowController
(socketDidAccept:): Change a -[refreshStatus] to an -[updateStatus]
(reconnect): Don't make a call on windowController
* Source/DebuggerController.m:
(init): Use the updated init method for GDBpConnection

Remove the model-updating-controller register variable code

* Source/GDBpConnection.m:
(refreshStatus): Removed
(run): Call -[updateStatus] and not -[refreshStatus]
(stepIn): ditto
(stepOut): ditto
(stepOver): ditto
(updateStackTraceAndRegisters): Removed
(createStackFrame): Fetch the contexts and variables
* Source/GDBpConnection.h: Header updates

Set the file column to truncate head, not truncate tail

* English.lproj/Debugger.xib

One more instance of calling -[setStatus:] that needs to change

* Source/GDBpConnection.m:
(reconnect): Change the ivar rather than calling setStatus

Rather than having GDBpConnection tell the window controller to change status, cache the status as an ivar and bind to it

* Source/GDBpConnection.h: Define the status ivar and make it a prop
* Source/GDBpConnection.m:
(refreshStatus): Remove most of the status-updating code
(updateStatus): New method to fetch and set the status ivar
* English.lproj/Debugger.xib: Bind the status text to GDBpConnection's new status ivar

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