Merge branch 'ui-polish'

Lightening all the toolbar icons, removing the shadows off the shapes, lightening the drop shadow, and adding the reconnect icon

Adding the Step Over and Run icons, as well as modifying the existing ones to be nicer

We were previously using, for the stack table, the index of the row to determine which source file to show instead of the "level" column of the row, which means sorting the stack table would screw things up

* Source/DebuggerWindowController.m:
([DebuggerWindowController updateSourceViewer]): Switch to using the selected item's "level" rather than the selected row index

Make sure that the source viewer is not editable

* Source/DebuggerWindowController.m:
([DebuggerWindowController awakeFromNib]):

We now properly draw the dimple in BSSplitView

* Source/BSSplitView.m:
([BSSplitView drawDividerInRect:]): Use -[NSImage compositeToPoint:operation:] instead of -[drawInRect:]

Draw the dimple (though it is upside-down because the Cartesian is flipped)

* dimple.png: Dimple image
* MacGDBp.xcodeproj: Project updates for the image
* Source/BSSplitView.m:
([BSSplitView drawDividerInRect:]): Add the dimple drawing code

Instead of using NSSplitView, use BSSplitView which is our custom implementation which draws a smaller, gradiated divider

* English.lproj/Debugger.xib: Switch to using a custom class for our two SplitView's
* Source/BSSplitView.h: New file
* Source/BSSplitView.m: ditto
* MacGDBp.xcodeproj: Project updates for new files

Adding some icons for the toolbar buttons (only Step In and Step Out)

* English.lproj/Debugger.xib: Set the icons for the two buttons
* Icons/StepIn.png: New file
* Icons/StepOut.png: ditto
* MacGDBp.xcodeproj: Project updates to add the icons to the resources
* Toolbar Buttons.psd: Original file for the icons

Polishing up the layout even more by removing some of the nasty double-border stuff

* English.lproj/Debugger.xib

Set springs on the split view and status text

* English.lproj/Debugger.xib

Making the split views go all the way to the edge, like in Xcode. Also, swap the registers/stack with the source view

* English.lproj/Debugger.xib

Use an NSToolbar for the debugger controls instead of NSButtons

* English.lproj/Debugger.nib: Removed because the designable NSToolbar requires an XIB
* English.lproj/Debugger.xib: Coverted old NIB
* MacGDBp.xcodeproj: Project updates for the nib->xib conversion
* Source/DebuggerWindowController.h: Changing the buttons to be of type NSButton to NSToolbarItem

The reconnect functionality now works

* Source/DebuggerConnection.h:
* Source/DebuggerConnection.m:
([DebuggerConnection reconnect]): New method that communicates the reconnect to the debugger
([Debuggerconnection refreshStatus]): If the status is "stopped", we are no longer connected
* Source/DebuggerWindowController.h:
* Source/DebuggerWindowController.m:
([DebuggerWindowController resetDisplays]): New method to set all the display information to be blank
([DebuggerWindowController reconnect:]): Forward the message to reconnect to the DebuggerConnection

Merge branch 'leopard'

The registers now have memory and stay expanded

* Source/DebuggerWindowController.h: expandedRegisters is now an NSMutableSet
* Source/DebuggerWindowController.m:
([DebuggerWindowController initWithPort:session:]): Change allocated type of expandedRegisters
([DebuggerWindowController setRegister:]): Finished logic for expanding remembered registers
([DebuggerWindowController outlineViewItemDidExpand:]): Use the "fullname" attribute to get the full variable name so scope is accounted for in expanded memory
([DebuggerWindowController outlineViewItemDidCollapse:]): ditto
* Source/NSXMLElementAdditions.h:
* Source/NSXMLElementAdditions.m:
([NSXMLElement(NSXMLElementAdditions) fullname]): New method

We now properly fetch registers who need to have more data attached

* Source/DebuggerWindowController.m:
([DebuggerWindowController addChildren:toNode:]): Don't attach nodes through the controller, but rather through the XMLElement (via the node's -[representedObject])

Removing the notification-delegate system that SocketWrapper used because it wasn't necessary because the only action that can block the UI is -[connect]

Changes:
-[SocketWrapper receive] now returns the data directly, cleaning up the DebuggerConnection class a lot
-[SocketWrapper send] returns BOOL depending on the success of the send

In SocketWraper, don't use the notification posting system to send errors to the delegate, instead just use performSelectorOnMainThread:withObject:waitUntilDone: on the delegate object. Also, -[NSObject(SocketWrapperDelegate) errorEncountered:] now just takes a string.

Synthesize new properties for class cross-referencing and SocketWrapper now init's with a DebuggerConnection

* Source/DebuggerConnection.h: New property for the DebuggerWindowController
* Source/DebuggerConnection.m: Synthesize new property and changed SocketWrapper's initializer
* Source/DebuggerWindowController.h: New property for DebuggerConnection
* Source/DebuggerWindowController.m: ditto
* Source/SocketWrapper.h: The init method now takes a DebuggerConnection (and stores it) instead of a port
* Source/SocketWrapper.m: ditto

Adding 2008 to the copyright year in the Info.plist and InfoPlist.strings files

Happy new year!

Use garbage collection and add -[SocketWrapper close] to close down the socket and not do it in -[finalize]

Merge branch 'refactoring' into leopard

Finish the refactoring in the previous commit and now the window title is properly set

* Source/DebuggerConnection.h: Removed the definition for -[windowDidClose]
* Source/DebuggerConnection.m:
([DebuggerConnection initWithWindowController:port:session:]): Don't set the status in this method because the nib isn't awake yet
* Source/DebuggerWindowController.m:
([DebuggerWindowController initWithPort:session:]): Make self key and front
([DebuggerWindowController awakeFromNib]): Set the default status to be "Connecting"
([DebuggerWindowController windowWillClose:]): Removed

We no longer use DebuggerConnection to manage the DebuggerWindowController, but now it's vice versa

Benefits:
- There was a problem that when we quit we would crash, because on window closing, the WindowController would tell the connection to shutdown, but because AppDelegate cleaned up before the WindowController, it would crash when sending the message.
- An array of connections no longer needs to be managed because the WindowController stores all the pointers, so when it closes things are deallocated properly

Instead of storing the pionter to the expanded item, save the variable name because we replace the entire register

Readd a space after a colon

Readd a space after a colon

Reworking the code to use NSTreeNode instead of the hidden proxy objects

We don't need the 10.5 SDK on the refactoring branch

Renaming the status ivar to be statusmsg so that -[DebuggerWindowController setStatus:] doesn't get called when connecting the nib

Refactoring _reconnectButton

Refactoring _runButton

Refactoring _stepOutButton

Refactoring _stepOverButton

Refactoring _stepInButton

Refactoring _sourceViewerScroller

Refactoring _sourceViewer

Refactoring _error

Refactoring _status

Refactoring _expandedRegisters

Refactoring _registerView

Refactoring _registerController

Refactoring _stack

Refactoring _stackController

Refactoring _connection ivar

Reverting portion of 48022096ee56de046900df9d3fedafe3e650c87b in DebuggerWindowController.m/h and Debugger.nib

Fix some variable hiding that was caused by our refactoring. Unfortunately, our code still hangs when we try to connect.

We can't have an instance variable match the name of a C function because then we don't compile

* Source/SocketWrapper.h: ivar socket is now sock
* Source/SocketWrapper.m:
([SocketWrapper connect:]): We're now happy because socket() can be called without ambiguity between self->socket

Untested refactoring is the best! Renaming all of the instance variables to not start with underscores, because it's just plain old annoying.

Removing the extra spacing after colons in function arguments

Add some test code to see how Leopard behaves

* MacGDBp.xcodeproj: Change to the 10.5 SDK
* Source/DebuggerWindowController.m: Play with the new and improved NSTreeController

Update MacGDBp.xcodeproj/.gitignore for Xcode v3

Minor upgrades to the project due to Xcode 3.0

Changing the version number from alpha to Beta 1

* Info.plist

Removing another NSLog when the stack's selection changes

* Source/DebuggerWindowController.m
([DebuggerWindowController tableViewSelectionDidChange:])

Improving the way we handle updating the stack and registers. Instead of updating them after we send step/run commands, only update the stacks and registers if the status (in updateStatus:) is on "break"

* Source/DebuggerConnection.m

Removing a puts statement from the shell script that creates the build version

* MacGDBp.xcodeproj

Removing the NSLog that was being called everytime we sent a command

* Source/DebuggerConnection.m
([DebuggerConnection dataSent:])

Renaming the "Refresh" butto to be "Re-Connect" and hooking it up (though it does nothing at the moment)

* English.lproj/Debugger.nib: Renaming the "Run" button and hooking it up to DebuggerWindowController
* Source/DebuggerWindowController.m|h: Adding a _reconnectButton outlet and a reconnect: action
([DebuggerWindowController reconnect:]): New method

Allow the connection socket to be reused in SocketWrapper

* Source/SocketWrapper.m:
([SocketWrapper connect:]): Allow the socketOpen to reuse addresses, making the bind errors less common

In the dataReceived: method if an error is in the response, set the error on the window

* Source/DebuggerConnection.m:
([DebuggerConnection dataReceived:deliverTo:]): If the response XML contains an error tag, set the error of the window to be the error message

Chaning all the delivery methods to accept NSXMLDocument instead of NSData and changing dataReceived: to create the NSXMLDocument

* Source/DebuggerConnection.m

We previously were leaking all over the place due to improper use of memory management techniques and circular references

* English.lproj/Debugger.nib: Setting DebuggerWindowController as the window's delgate
* Source/AppDelegate.h: Adding a connections ivar
* Source/AppDelegate.m:
([AppDelegate init]): New method to initialize the connections array
([AppDelegate applicationWillTerminate:]): Frees the connections array
([AppDelegate registerConnection:]): New method to register a DebuggerConnection
([AppDelegate unregisterConnection:]): Unregisters a DebuggerConnection
* Source/ConnectWindowController.m:
([ConnectWindowController connect:]): We weren't properly release'ing the connection, so now we register it with the delegate and release it in the connect method
* Source/DeubggerConnection.h: Added a windowDidClose method so that the window controller can forward the message
* Source/DebuggerConnection.m:
([DebuggerConnection windowDidClose]): New method to forward the windowWillClose notification
([DebuggerConnection initWithPort:session:]): We no longer have to register for NSApplicationWillTerminate
([DebuggerConnection applicationWillTerminate:]): Removed
([DebuggerConnection dealloc]): Release _windowController
* Source/DebuggerWindowController.m:
([DebuggerWindowController initWithConnection:]): No longer retain the connection as it creates a circular ref
([DebuggerWindowController windowWillClose:]): Use this delegate method to buble the notification to the controller
([DebuggerWindowController dealloc]): Since we don't retain _connection, don't release it
* Source/SocketWrapper.m:
([SocketWrapper dealloc]): Remove ourselves as a notification observer so we don't have a nil reference

We now remember which items in the register have been expanded so they stay open

* Source/DebuggerWindowController.h: Adding an _registerView outlet and an _expandedRegisters array
* Source/DebuggerWindowController.m:
([DebuggerWindowController initWithConnection:]): Initialize _expandedRegisters
([DebuggerWindowController dealloc]): Release _expandedRegisters
([DebuggerWindowController setRegister:]): When we set the register, go through all the items and see if any of them should be expanded
([DebuggerWindowController outlineViewItemDidExpand:]): Add the expanded object to _expandedRegisters
([DebuggerWindowController outlineViewItemDidCollapse:]): Remove the object from _expandedRegisters
* English.lproj/Debugger.nib: Hooking up the _registerView outlet

Working around an extremely annoying apple bug in NSTreeController and an NSOutlineView. When the contentArray is bound on an NSTreeController, the observing can go haywire and just crash randomly.

* English.lproj/Debugger.nib: Unbinding RegisterController's content array
* Source/DebuggerConnection.m:
([DebuggerConnection registerRecieved:]): Sending [_windowController setRegister:] the entire document instead of the rootElement
* Source/DebuggerWindowController.h: Removed _register ivar and changed setRegister: to take an NSXMLDocument
* Source/DebuggerWindowController.m:
([DebuggerWindowController setRegister:]): Instead of changing a property and letting KVO do the work (and subsequently and randomly crash), force the controller to have new content

Casting to remove a warning

* Source/DebuggerConnection.m:
([DebuggerConnection propertyRecieved:]): Casting parent to be an NSXMLElement because childAtIndex returns an NSXMLNode

Making the buttons change their enabled state depending on where we are in debugging

* Source/DebuggerWindowController.m:
([DebuggerWindowController setStatus:]): Disable all the buttons and reenable "step in" and "run" if we're connected
([DebuggerWindowController setStack:]): Enable the step in/out/over and run buttons appropriately

Hooking up the step in/out/over and run buttons to the nib

* Source/DebuggerWindowController.h: Making outlets for all of the buttons
* English.lproj/Debugger.nib: Hooking up the outlets to the interface

Adding a method to check and see if the connection we have is valid

* Source/DebuggerConnection.m|h:
([DebuggerConnection isConnected]): New method

Adding an Apple radar number for where we have to use _NSArrayControllerTreeNode

* Source/DebuggerWindowController.m:
([DebuggerWindowController outlineViewItemDidExpand:])
([DebuggerWindowController addChildren:toNode:])

Commenting out the data received NSLog()

* Source/SocketWrapper.m:
([SocketWrapper receive:])

The depth fetching of child nodes now works and are properly inserted into the register tree

* Source/DebuggerConnection.h: Changing _depthFetchElement to type id for the mysterious _NSArrayControllerTreeNode type
* Source/DebuggerConnection.m:
([DebuggerConnection propertyReceived:]): Get all the children from the property and then detach them so they can be injected into the tree
* Source/DebuggerWindowController.h: Added a _registerController outlet
* Source/DebuggerWindowController.m:
([DebuggerWindowController addChildren:toNode:]): New method that adds detached children to the specified _NSArrayControllerTreeNod using it's index set
([DebuggerWindowController outlineViewItemDidExpand:]): Change from using an NSXMLElement to the internal TreeNode
* English.lproj/Debugger.nib: Hook up the RegisterController to the outlet

Fixing a compiler error for a missing variable

* Source/DebuggerConnection.m:
([DebuggerConnection getProperty:forElement:]): We were still referencing a depth variable when we removed it

We now get the property when we go past the currently-fetched depth but we do not have the display updating yet

* Source/DebuggerConnection.h: New ivar _depthFetchElement to record the element that requested more registers
* Source/DebuggerConnection.m:
([DebuggerConnection dataReceived:deliverTo:]): Removed an NSLog()
([DebuggerConnection getProperty:forElement:]): New method that gets a specified property for a specific element
([DebuggerConnection propertyReceived:]): Recived data delivery handler for getProperty:forElement:
* Source/DebuggerWindowController.m:
([DebuggerWindowController outlineViewItemDidExpand:]): Call the getProperty:forElement: method to get deeper properties
* English.lproj/Debugger.nib: Set the delegate of the NSOutlineView to be the first responder

Moving the depth checker from NSXMLElementAdditions.m to be in DebuggerWindowController

* Source/NSXMLElementAdditions.m:
([NSXMLElement isLeaf]): Moved the checking to see if we need to fetch more
* Source/NSXMLElementAdditions.h: New header file for linking
* Source/DebuggerWindowController.m:
([DebuggerWindowController outlineViewItemDidExpand:]): New method that handles the depth checking
* MacGDBp.xcodeproj: Project changes for NSXMLElementAdditions.h

Adding support for base64 encoded string values

* Source/base64.c|h: New file from external source
* Source/NSXMLElementAdditions.m:
([NSXMLElement value]): Decode data using the new base64 functions
* MacGDBp.xcodeproj: Adding the base64.c|h files to the project

Merge branch 'master' into register-feature

Conflicts:

MacGDBp.xcodeproj/project.pbxproj

Reorganizing the project so classes are actually int he classes group and not in "Other Sources"

In NSXMLElementAdditions adding recognizers to the code for certain special value types

* Source/NSXMLElementAdditions.m:
([NSXMLElement value]): If the item is a leaf, only display an elipsis, and if it's base64 encoded the decode.. er print that it's base64'd

Making the register viewer have a smaller font

* English.lproj/Debugger.nib

Make a cast to remove a warning

* Source/NSXMLElementAdditions.m:
([NSXMLElement leaf]): Cast [elm parent] to be NSXMLElement to remove a warning

Removing some NSLog()s and in isLeaf: we now determine if a non-leaf node has it's children

* Source/NSXMLElementAdditions.m:
([NSXMLElement variable]): Remove a NSLog()
([NSXMLElement isLeaf]): If the element is not a leaf, but does not have children, then find the depth we should request more about the property at

Standardizing the NSXMLElementAdditions methods and adding a new one for variable type

* Source/NSXMLElementAdditions.m:
([NSXMLElement varName]): Renamed to variable
([NSXMLElement type]): New method

Merge branch 'master' into register-feature

Use memset() on some of the char[]'s to make sure we don't have problems with memory

* Source/SocketWrapper.m:
([SocketWrapper receive:]): Set the packetLength char array and the packet char array to be null before writing into them

Trying to get the register viewer to work

* Source/NSXMLElementAdditions.m: New category that provides access to attributes that NSTreeController can't access
* MacGDBp.xcodeproj: Added NSXMLElementAdditions.m
* Source/DebuggerConnection.m:
([DebuggerConnection dataReceived:deliverTo:]): Print out the data we're receiving to make debugging easier
([DebuggerConnection updateStackTraceAndRegisters:]): Send the context_get command and receive the data
([DebuggerConnection registerReceived:]): New method to handle the received register packet
* Source/DebuggerWindowController.m|h: Added _register and a setter for it
([DebuggerWindowController setRegister:]): New method
* English.lproj/Debugger.nib: Added a tree controller and tried to hook it up properly

Fixing the updateSourceViewer function to not depend on -[NSArrayController selection] which is unreliable in returning the right proxy object

* Source/DebuggerWindowController.h: Remove _currentFile as it's not really needed
* Source/DebuggerWindowController.m:
([DebuggerWindowController updateSourceViewer]): Remove the dependency on [NSArrayController selection] and don't bother checking with _currentFile anymore

Scroll to the proper line in the source file and colorize it when we are clicking through the stack trace

* Source/DebuggerWindowController.m

Calling updateSourceViewer: on selection change isn't enough, we also need to do it when the stack changes

* Source/DebuggerWindowController.m:
([DebuggerWindowController setStack:]): Update the source viewer after finishing the setting

Styling the NSTextView source viewer to be a fixed-width font and to have a horizontal scroll bar

* Enblish.lproj/Debugger.nib: Adding an outlet for the NSScrollView and setting some NSTextView options
* Source/DebuggerController.m:
([DebuggerWindowController awakeFromNib]): New method to set up the scrolling system for the source viewer
([DebuggerWindowController updateSourceViewer]): Set the string to "" if there is no selected file, and make the font Monaco

Starting to implement the source viewer pane

* Source/DebuggerWindowController.h: Adding outlets for StackController, _currentFile, and _sourceViewer
* Source/DebuggerWindowController.m:
([DebuggerWindowController tableViewSelectionDidChange:]): Added delegate notification for when the selection of the tableview changes to update the source viewer
([DebuggerWindowController updateSourceViewer]): New method to refresh the source viewer display
* English.lproj/Debugger.nib: Hooking up the new outlets

I guess we forgot to rename _postNotification to postNotification, too

* Source/SocketWrapper.m

Forgot to change the instances of _createCommand to createCommand

* Source/DebuggerConnection.m

Cleaning up our use of private methods: Moving all private methods into class categories in the .m file, and removing the _ prefix as it is "reserved" for Apple (according to their guidelines).

Hooking up "Step Over" and "Step Out" buttons

* Source/DebuggerConnection.m|h:
([DebuggerConnection stepOut]): New method
([DebuggerConnection stepOver]): New method
* Source/DebuggerWindowController.m|h:
([DebuggerWindowController stepOut]): New method
([DebuggerWindowController stepOver]): New method
* English.lproj/Debugger.nib: Hooking up the two buttons to the new actions in DebuggerWindowController

Now that we use NSData, it doesn't make sense to print out the received packet

* Source/DebuggerConnection.m:
([DebuggerConnection dataReceived:deliverTo:])

Instead of data received being an NSString, have it be NSData because converting to NSString was actually causing problems and NSXMLDocument can take in NSData just fine

* Source/DebuggerConnection.m: Making all of the delivery functions use NSData not NSString
([DebuggerConnection _handshake:])
([DebuggerConnection _updateStatus:])
([DebuggerConnection _stackReceived:])
* Source/SocketWrapper.m|h: ditto
([SocketWrapper dataReceived:deliverTo:]): Now uses NSData
([SocketWrapper receive:]): Save ourselves some trouble and don't convert the data to NSString

Revert "Instead of directly calling stack_get, get the depth first and then call the maximum depth"

This reverts commit 9371ab6efb4d371e9db5f9e72b8f2cdf38a3a8de.