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

Adding IBAs to Breakpoints.xib and the BWC

* English.lproj/Breakpoints.xib: Adding actions for the +/- buttons
* Source/BreakpointWindowController.m+h: Adding the code complements for the IBAs

Rewrite BreakpointManager to use NSArray instead a faux hashtable using a NSDictionary

* Source/Breakpoint.m+h

Add an ivar for the BreakpointManager in the BreakpointWindowController

* Source/BreakpointWindowController.h: Add the ivar manager
* Source/BreakpointWindowController.m:
(init:): Set the ivar to the sharedManager

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

Fixed a warning in Source/BSSourceView.m:48 -- method possibly missing a [super dealloc] call

Disabling GC

* MacGDBp.xcodeproj

Merge branch 'master' into no-gc

Conflicts:

Source/Breakpoint.m
Source/ConnectWindowController.m

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

Turning off GC and adding back manual memory management

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