Don't drain the pool, just release it.

Lazily load the complete stack frame for anything but the current one.

* In StackController, don't update the frame indices when |-push:|- or |-pop|-ing.
* Only retreive full stack frame information for frame 0. The other frames will be fetched lazily.

Give |-connectInternal| an autorelease pool.

Re-organize DebuggerConnection to give it some shape and a decent public interface.

Remove objects from the |callTable_| after handing the response.

Put a lock around adding to the write queue in |-send|.

Make DebuggerConnection work on a separate thread.

Move the callTable back into DebuggerProcessor.

Finished enough of the basic refactoring to get things working.

Add a new DebuggerConnection that separates out the socket logic from the handling logic.

This compiles, but will most certainly not work. This is just the initial separation of
methods and ivars that needs to be rewired.

Rename DebuggerConnection to DebuggerProcessor in preparation for the separation between the two.

Don't |-send:| if the transactions do not match

Replace strlen() with a loop that we can guarantee.

Rewrite |-readStreamHasData| in an effort to make it more robust.

Fix some potential bugs and do general cleanup.

Rewrite DebuggerConnection's packet handling sytem. The initial version failed
to account for multiple packets in the same stream read.

* Rewrite |-readStreamHasData| to just parse out string packets
* Created |-handlePacket:| to perform the XML parsing and dispatching

* Factor out some code for parsing the txnID out of a command into |-transactionIDFromCommand:|.
* Change the way we use the write queue, with the aim of producing less into it and starving it infrequently.

Use the new VersionCast URL.

Remove duplicated logic.

Try to recover from erroneous XML packets.

* Do not allow editing of the log table.
* Move the transaction checks from |-handleResponse:| to |-readStreamHasData|.
* Record log entries AFTER sending and receiving so that the transaction IDs make more sense.

Officially kill TransportDebug env setting.

Create a LoggingController that shows all the commands sent and received.

This will replace the TransportDebug variable. This first pass just adds very
basic support for this functionality. The view still needs some work.

Create a top-level dev/ directory. Move resource masters there and import CFNetworkTest.

Don't crash after pressing the "run" button or at end of script.

Remove two obsolete methods: |-[DebuggerConnection getCurrentStack]| and |-[DebuggerController reloadStack]|.

Rewrite |-[DebuggerConnection getProperty:]| to be asynchronous.

Try to recover from malformed XML errors by manually parsing out the transaction_id.

Rename GDBpConnection to DebuggerConnection. Also organize all the ivars and methods.

Switch some basic ints to NSUInteger in GDBpConnection.

Switch from GCC to Clang compiler.

* Start plumbing data back to the UI now that we're partially working asynchronously
* Set up a callback for |-addBreakpoint:|
* Create |-transactionIDFromResponse:| helper

Receive contexts and then request their variables; finish rewriting |-createStackFrame:|.

Ditch the hybrid routed/command dispatcher system for a pure callback system.

Everything is still broken, but this change did not regress anything further.

Fix encoding issues by not implicit casting UInt8 to char. Go through CFString instead.

Always clobber the stack in |-debuggerStep:|

Use a lock to make |-sendQueuedWrites| a critical section

* Start tracking the last read and last written transaction IDs
* Start numbering transactions at 1, rather than 0
* Kill some NSLogs
* Try to enfore strict ordering of read/writes using the last transaction information

* Start making |-createStackFrame:| asynchronous
* Give StackFrame objects a routingID_
* Queue writes in GDBpConnection

Happy new year!

Xcode project updates:
* Make Gitcrement less annoying to work with
* Xcode 3.2 updates

* Create functionality to perform non-blocking writes using a write queue
* Fix up the debugger step commands to use async communication

Remove SocketWrapper and start using CFSocket with CFStreams in GDBpConnection. This compiles but not much else.

Prevent the Preferences window from jumping around when it opens by disabling
the "Visible on Launch" property.

Remove IDE key preference because this is a property from the engine, not the IDE.

Remove the |session| ivar from GDBpConnection. Use |transactionID| to keep
track of individual message transactions with the debugger engine.

Make our project Xcode 3.1+ compatible.

Change NSXMLElementAdditions to use UTF8 strings when converting, rather than ASCII.

Make the transport logging an environmental setting.
To enable: `setenv TransportDebug YES`.

In |-debuggerDisconnected|, unmark the program counter line so it doesn't look like we're connected.

Fix warnings about GDBpConnectionDelegate by including the header in DebuggerController.h.

Version to 1.4b.

* Add a preference to automatically reconnect. Fixes bug #165.
* Create a GDBpConnectionDelegate formal protocol.
  * No longer use notifications to pass errors from GDBpConnection to DebuggerController.
  * Add delegate methods for |-debuggerConnected| and |-debuggerDisconnected|.
* Move GDBpConnection's |-socketDidAccept| implementation into a helper so it can be run on the main thread.

* Completely de-couple SocketWrapper from GDBpConnection by passing the port directly to the ctor, rather than extracting it from the connection.
* Make SocketWrapperDelegate a formal protocl and GDBpConnection now conforms to it.
* Clean up delegate interface.

Add a tooltip to the "break on first line" preference.

Switch to using -[NSString stringWithUTF8String:] in SocketWrapper.

Mark one of the fixes as bug 157

Add a toolbar icon for the HUD window, which required making DebuggerController the action handler.

* Icons/HUDIcon.png: New icon
* CONTRIBUTORS: Add credits for icon
* MacGDBp.xcodeproj/project.pbxproj: Project updates for icon
* Source/AppDelegate.h+m:
(showInspectorWindow:): Moved to...
* Source/DebuggerController.h+m:
(showInspectorWindow:): new from ^
* English.lproj/Debugger.xib: Add toolbar item
* English.lproj/MainMenu.xib: Change Window-->Inspector's target

We need to use UTF8 string encoding instead of ASCII because we're receiving XML. Socket code should be perfect now.

* Source/SocketWrapper.m:
(receive)

Put the debugger behavior settings above the Xdebug ones

* English.lproj/Preferences.xib

Rewrote -[SocketWrapper receive] so that we do not get any more partial/broken packets, hopefully.

* Source/SocketWrapper.m:
(receive)

More fixes for paths with spaces. We can't use single quotes to pass path names, so create a new method to escape them.

* Source/GDBpConnection.m:
(escapedURIPath): New private method that takes a path and returns a safe file:// URI
(addBreakpoint:): Use -[escapedURIPath:]
(createStackFrame:): Don't pass the escaped path to the frame, keep it internally

Don't try to manage the stack using internal state anymore, simply grab it every time from the engine

* Soure/DebuggerController.m:
(stepIn:): The GDBpConnection methods no longer return anything, get entire stack now
(stepOut:): ditto
(stepOver:): ditto
* Soure/GDBpConnection.h+m:
(run): Return void
(stepIn): ditto
(stepOut): ditto
(stepOver): ditto

Add a preference to disable automatic step-in upon connection

* English.lproj/Preferences.xib: Add checkbox to the general prefs
* Source/AppDelegate.m: Add NSUserDefaults default value
* Source/DebuggerController.m:
(startDebugger): Check pref before stepping

Fix a type mismatch compile error

* Source/GDBpConnection.h: Change NSArray to StackFrame for -[run]

Merge branch 'bug-157'

Conflicts:
Source/DebuggerController.m
Source/GDBpConnection.m

Merge branch 'cosmetics'

Conflicts:
CHANGES
Source/GDBpConnection.m

Factored out the stack reloading code

* Source/DebuggerController.m:
(reloadStack): New private method to fetch and insert new stack
(run): Use reloadStack
* Source/GDBpConnection.m+h:
(getCurrentStack): New method to get the stack and create all the stack frames
(run): Return only the current stack frame instead of a whole stack

Add NSLog()s around the send/receive stuff so we can track activity with Xdebug

* MacGDBp.xcodeproj: In debug mode, define BLU_DEBUG as a C flag
* Source/GDBpConnection.m:
(createCommand:)
(processData:)

When getting the source, we need to escape the '%20' space character so the source displays properly

* Source/GDBpConnection.m:
(createStackFrame)

Breakpoints with a space in the filename would not be set.

* Source/GDBpConnection.m:
(addBreakpoint:): Quote the breakpoint path before sending the GDBp command

Make the asterisk attached to the type, rather than the variable.

Fix the run command by having it obliterate the current stack and set a new one

* Source/GDBpConnection.h+m:
(run): Now returns an NSArray of the new stack frames
(createStackFrame): Sets the StackFrame's index to be the depth
* Source/DebuggerController.m:
(run:): Set the new stack and refresh the source viewer

Change -[createStackFrame] to take a depth and create private -[createCurrentStackFrame] to call it with depth 0

* Source/GDBpConnection.m:
(createStackFrame:): Take a new depth argument
(createCurrentStackFrame): New method to call above with 0

Set the version to be 1.3

* Info.plist

Set the SourceView.file to nil and tell the stack NSArrayController to rearrange

* Source/DebuggerController.m:
(resetDisplays)

Make -[GDBpConnection(Private) createCommand:] take a variable number of arguments to parse

* Source/GDBpConnection.m:
(createCommand:)

Before setting selectedVariable, we need to make sure there is a selection!

* Source/DebuggerController.m:
(stepIn:)
(stepOut:)
(stepOver:)

Retain the selection in the variable list when stepping through code

* Source/DebuggerController.h: Add the selectedVariable ivar
* Source/DebuggerController.m:
(stepIn:): Save selected variable
(stepOut:): ditto
(stepOver:): ditto
(expandVariables): When looping over variables to expand, also reset the selection

NSLog() when Xdebug returns us an error

* Source/GDBpConnection.m:
(processData:)

Not all breakpoint markers would be displayed in the active code debugger

* Source/DebuggerController.m:
(updateSourceViewer): Get all the breakpoints for the file and set the markers

Update the changelog

Have the inspector remember its visibility

* Source/AppDelegate.m:
(load): Set the default value
* Source/DebuggerController.m:
(init): Show/hide the inspector based on prefs
* English.lproj/Debugger.xib: Bind the inspector's visibility

When toggling the breakpoints window, we also need to check to make sure if it's visible but not key

* Source/AppDelegate.m:
(showBreakpointWindow:): If the window is visible but not key, order it front

Add an item to the Window menu to allow the inspector to be shown/hidden

* Source/AppDelegate.m+h:
(showInspectorWindow:): New method
(showBreakpointWindow:): Make this toggle window visibility rather than just always bring to front
* Source/DebuggerController.m+h: Create ivar for the inspector and synth the property
* English.lproj/Debugger.xib: Hook up the outlet
* English.lproj/MainMenu.xib: Add menu item

Add a variable value inspector

* English.lproj/Debugger.xib: Needed to remove the "Content" binding to make this work. Added a foating HUD window.

Update version to 1.2.1

* Info.plist

Add the public key for Sparkle updates

* Info.plist: Add the key file entry
* dsa_pub.pem: New file
* MacGDBp.xcodeproj: Project updates for keyfile

Update Sparkle to be a vanilla build now that we properly use CFBundleVersion

Update the project to use Gitcrement versioning

* MacGDBp.xcodeproj

Updated Sparkle to the latest bzr version

Fixed a segfault crash that would occur if you stepped at the end of a script

* Source/GDBpConnection.m:
(stepIn): Update the status before creating the stack frame so we know if we're connected or not
(stepOut): ditto
(stepOver): ditto

When writing the temporary file to use for highlighting, use NSUTF8StringEncoding rather than NSASCIIStringEncoding

* Source/BSSourceView.m:
(setString:asFile:)

Don't include the final .0 in 1.2

* Info.plist

Convert the run command to use the StackFrame system

* Source/GDBpConnection.m+h:
(run): Now returns a StackFrame object
(processData:): Return nil if the given data is nil
(createStackFrame): If we get an empty NSXMLDocument for the stack, return nil
(updateStatus): If the status is nil, the debugger is stopped
* Source/DebuggerController.m:
(run): Use the stack frame
* Source/SocketWrapper.m:
(receive): If the call to recv() returns -1 (error), return nil string

Merge branch 'path-replacement'

Conflicts:
CHANGES
Source/AppDelegate.m

The breakpoints window will now remember if it was opened or closed

* English.lproj/Breakpoints.xib: Bind the visible status to NSUserDefaults
* Source/AppDelegate.m:
(load): Add the default value for BreakpointsWindowVisible stdudef
* Source/BreakpointController.m:
(init): Only order the window back if BreakpointsWindowVisible is true

Bumping the version number to 1.2.0

* Info.plist

Add a changelog entry for the path replacements prefs