Fixing some UI quirks in the breakpoints window

* English.lproj/Breakpoints.xib

Finishing hooking up the XIB

* English.lproj/Breakpoints.xib: Set File's Owner to be BreakpointWindowController and set the window IBO
* Source/BreakpointWindowController.m:
(init): Adding a comment

We need an instance of BreakpointWindowController in AppDelegate

* Source/AppDelegate.m:
(applicationDidFinishLaunching:): Init BreakpointWindowController to breakpoints
* Source/BreakpointWindowController.m:
(init): Add an init method

Adding a breakpoints NSWindowController for Breakpoints.xib

* Source/BreakpointWindowController.m+h: New files
* MacGDBp.xcodeproj: Project updates for BreakpointWindowController
* Source/AppDelegate.m+h:
(showBreakpointWindow:): New method
* English.lproj/MainMenu.xib: Add a

Adding Breakpoints.xib for the breakpoints manager

* English.lproj/Breakpoints.xib: New file
* MacGDBp.xcodeproj: Project updates for the new XIB

We aren't going to store breakpoints in NSUserDefaults

* Source/BreakpointManager.m:
(init): Removed the NSUserDefaults loading
(addBreakpoint:): Don't add it to NSUserDefaults
(removeBreakpoint:): ditto

Adding NSDictionary archiving and unarchiving abilities to Breakpoint

* Source/Breakpoint.m+h:
(initWithDictionary:): New method
(dictionary): New method

Reorganizing the project source files into groups

* MacGDBp.xcodeproj

When calling -[SocketWrapper remoteHost], we don't want to continually query the hostname

* Source/SocketWrapper.h: Adding a hostname ivar
* Source/SocketWrapper.m:
(remoteHost): Just return the hostname ivar
(connect:): Query the hostname and store it in the hostname ivar

Sending the "run" command was balanced with a receive command

* Source/DebuggerConnection.m:
(run): Message the socket to -[receive]

Comment out the saving of breakpoints to NSUserDefaults because it's generating a lot of noise

* Source/BreakpointManager:
(addBreakpoint:)
(removeBreakpointAt:inFile:)

Make the stack text be the same size as the register text

* English.lproj/Debugger.xib

We don't want the NULL byte at the end of the NSString when we call -receive:

* Source/SocketWrapper.m:
([receive]): Substring to the last byte

Instead of using NSData for [SocketWrapper receive], use NSString

* Source/DebuggerConnection.m:
([processData:]): Use NSString instead of NSData
* Source/SocketWrapper.m+h:
([receive]): Return an NSString

Putting GC back on because we really need it

Forgot to turn off GC in debug mode

* MacGDBp.xcodeproj

Add better error checking for malformed XML

* Source/DebuggerConnection.m:
(-[processData:]): If there's an XML error, print a LOT of info about it

Trying to add some bounds checking so we don't get unhandled exceptions

* Source/DebuggerConnection.m:
(-[processData:]): Try to catch malformed data
* Source/DebuggerWindowController.m:
(-[updateSourceViewer]): Don't proceed if there is no stack

Don't try to load empty files (like when we're in eval())

* Source/DebuggerWindowController.m:
(-[updateSourceViewer]): If the filename is empty, abort the data refresh

Making a reference to the debugger in the Window menu

* English.lproj/MainMenu.xib: Add a Window->Debugger item
* Source/AppDelegate.m/h: Add a debugger ivar to hold the DebuggerWindowController
([showDebuggerWindow:]): New method to bring the debugger window to the front
* Source/DebuggerWindowContrller.m:
([awakeFromNib]): Exclude the window from the dynamic Window menu

Removing the connect dialog

* English.lproj/Connect.xib: Removed
* English.lproj/MainMenu.xib: Removing the "Connection" menu and adding a "File" menu
* Source/AppDelegate.m/h:
([showConnectWindow:]): Removed
* Source/ConnectWindowController.m/h: Removed
* MacGDBp.xcodeproj: Project updates for removed files

Add an initial breakpoints manager window and save the breakpoints to NSUserDefaults

Merge branch 'breakpoints'

When clicking a breakpoint that exists, tell the connection to remove it

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

Return the breakpoint when removing in BreakpointManager

* Source/BreakpointManager.m/h:
([BreakpointManager removeBreakpointAt:inFile:]): Return the breakpoint that was removed

Adding support to actually set breakpoints on the debuger side

* Source/Breakpoint.h: New debuggerId ivar to keep track of the engine's tracking ID
* Source/Breakpoint.m: Synthesize ivar
* Source/DebuggerConnection.m/h:
([DebuggerConnection addBreakpoint:]): New method
([DebuggerConnection removeBreakpoint:]): New method
* Source/DebuggerWindowController.m:
([DebuggerWindowController gutterClickedAtLine:forFile:]): When adding a breakpoint, tell the engine to set it

Add support for removing a breakpoint

* Source/BSLineNumberView.m:
([BSLineNumberView drawRect:]): Remove an NSLog()
* Source/BreakpointManager.m:
([BreakpointManager removeBreakpointAt:inFile:]): New method
* Source/DebuggerWindowController.m:
([DebuggerWindowController gutterClickedAtLine:forFile:]): Add support for removing breakpoints

Change the drawing of the marker to make it prettier

* Source/BSLineNumberView.m:
([BSLineNUmberView(Private) drawMarkerInRect:]): Make the mark darker and stroked

Add the functionality to draw markers

* Source/BSLineNumberView.m:
([BSLineNumberView(Private) drawMarkerInRect:]): New method

Adding a new class called BreakpointManager so we don't clutter up AppDelegate

* Source/AppDelegate.m/h: Removing the breakpoint code
* Source/BSLineNumberView.m:
([BSLineNumberView mouseDown:]): We need to add the lineNumberRange so we get an accurate number
* Source/DebuggerWindowController.m: Switching to BreakpointManager instead of AppDelegate for managing breakpoints
* Source/BreakpointManager.m/h: New files
* MacGDBp.xcodeproj: Project updates for BreakpointManager

Initial breakpoint adding code

* MacGDBp.xcodeproj: Project updates for Breakpoint.m/h
* Source/AppDelegate.h: Add a breakpoints ivar
* Source/AppDelegate.m:
([AppDelegate addBreakpoint:]): New method
([AppDelegate breakpointsForFile:]): New method
* Source/BSLineNumberView.h: Add a markers ivar
* Source/BSLineNumberView.m:
([BSLineNumberView drawRect:]): Add breakpoint-detecting code
* Source/DebuggerWindowController.m: Making this the delegate of BSSourceView
([DebuggerWindowController gutterClickedAtLine:forFile:]): Implementing delegate method

We now properly handle the mouseDown event in the gutter

* Source/BSLineNumberView.h: New property called lineNumberRange
* Source/BSLineNumberView.m: Synthesize lineNumberRange and set it to be empty in -[init]
([BSLineNumberView drawRect:]): Set lineNumberRange to be the range of visible lines in the source viewer
([BSLineNumberView mouseDown:]): Use the line number range to simplify the code of determining which line was clicked
* Source/BSSourceView.h: New property file to hold the filename; also create delegate category
* Source/BSSourceView.h: Synthesize file
([BSSourceView setFile:]): Sets the file property as well as setting the string of the textView object
* Source/DebuggerWindowController.m:
([DebuggerWindowController updateSourceViewer]): Use the new -[setFile:] method

Add an application icon

* Icons/MacGDBp.icns: New icon file
* Info.plist: Add a reference to the icon file
* MacGDBp.xcodeproj: Project updates for the icon file
* Resource Masters/App Icon.psd: New file
* Resource Masters/Globe.ai: New file
* Resource Masters/Toolbar Buttons.psd: Moved from root folder

Merge branch 'master' into breakpoints

Allow the type to also display the class of object

* Source/NSXMLElementAdditions.m:
([NSXMLElement(NSXMLElementAdditions) type]): If the classname attribute is present, use it

The mouse down handler almost works, but our math is still off

* Source/BSLineNumberView.m:
([BSLineNumberView mouseDown:])

Making progress on our -[mouseDown:] event handler, but it's not quite working

* Source/BSLineNumberView.m:
([BSLineNumberView mouseDown:]): New method
* Source/BSSourceView.h: New delegate prop-ivar
* Source/BSSourceView.m: Synthesizing the delegate ivar

We now highlight the break'd line

* Source/BSSourceView.h: New prop-ivar markedLine
* Source/BSSourceView.m: ditto
* Source/BSSourceViewTextView.m:
([BSSourceViewTextView drawRect:]): Highlight the line
* Source/DebuggerWindowContrller.m:
([DebuggerWindowController updateSourceViewer]): Call [sourceViewer setMarkedLine:]

Adding a method to scroll to a given line in the BSSourceView

* Source/BSSourceView.m:
([BSSourceView scrollToLine:]): New method that scrolls to a specific line in the NSTextView
* Source/BSSourceView.h: ditto
* Source/DebuggerWindowController.m:
([DebuggerWindowController updateSourceViewer]): Removing the scrolling logic that used to be here

Set springs on the debugger window

* English.lproj/Debugger.xib

We now draw the line number in the the middle of the the line's height, which means that if the line number is a smaller size than the line text it will be centered instead of drawn at the top

* Source/BSLineNumberView.m:
([BSLineNumberView drawRect:])

Stroke the gutter at the top and bottom

* Source/BSLineNumberView.m:
([BSLineNumberView drawRect:]): Draw a line at the top and bottom of the gutter view

Fixing a memory leak

* Source/NSXMLElementAdditions.m:
([NSXMLElement(NSXMLElementAdditions) value]): base64_decode_alloc() calls malloc() but not free() so we need to do it ourselves

Add a bunch of new NSView classes (to form a meta-view BSSourceView) that adds a line-numbered NSTextView embedded in a NSScrollView

* English.lproj/Debugger.xib: Use our BSSourceView instead of an NSTextView for the source viewer
* Source/BSLineNumberView.h: new file
* Source/BSLineNumberView.m: new file
* Source/BSSourceView.h: new file
* Source/BSSourceView.m: new file
* Source/BSSourceViewTextView.h: new file
* Source/BSSourceViewTextView.m: new file
* MacGDBp.xcodeproj: Project updates for all the new files
* Source/DebuggerWindowController.h: Switch to using a BSSourceView
* Source/DebuggerWindowController.m: ditto

The icons all had a bit of highlight hanging in the alpha, so removed all of it

* Icons/Reconnect.png
* Icons/Run.png
* Icons/StepIn.png
* Icons/StepOut.png
* Icons/StepOver.png
* Toolbar Buttons.psd

Make the register and stack tables have uniform column resizing

* English.lproj/Debugger.xib

Converting the rest of the NIBs to be XIBs simply for the sake of consistency

* English.lproj/Connect.nib: Removed
* English.lproj/MainMenu.nib: Removed
* English.lproj/Connect.xib: New file
* English.lproj/MainMenu.xib: New file
* MacGDBp.xcodeproj: Project updates for removed NIBs and new XIBs, also put all the XIBs into a group for organization sake

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