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

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