Add a quit source to the DebuggerConnection's run loop to properly handle closing.

Properly set the |connected_| flag in the Connection to NO when we're stopped. Prevents an EXC_BAD_ACCESS on |-run:|.

Rename |-connectInternal| to |-connectionThreadStart|.

Remove 2 super verbose NSLog() statements.

Switch to using VariableNode in the interface. Reduce use of NSXMLElementAdditions to just the key methods.

Add VariableNode which will replace our use of raw NSXMLElement.

Remove the pending property after it is received.

Initial work to get properties loading asynchronously. This may have a memory corruption bug at the moment...

Add a singleton |+instance| method to AppDelegate so we don't cast everywhere.

Stop crashing with EXC_BAD_ACCESS when expanding properties beyond the fetched depth.

Make logging threadsafe.

Tabs to spaces.

Begin making DebuggerConnection more threadsafe.

Xcode project file updates.

Make StackFrame a dumb data structure and remove its initializer. All properties are now read-write.

This fixes two warnings produced by Clang-SA and avoids sticking an |+alloc|ed but non-
|-init...|ed class inside the |stackFrames_| array.

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