We need to set self.attached before referencing it.

Remove one more NSLog.

Remove some NSLog statements

Close and reopen the connection when toggling the attached state.

Move dealing with the socket into NetworkCallbackController.

Remove beta moniker and unstable info from Info.plist.

Remove NSLog()

Add code to the AppDelegate to record if a user has ever used an unstable VersionCast feed

Remove some dead code in -[NetworkConnection socketDisconnected]

Fix a typo

The attached state is now persisted to preferences.

Update changelog

Disallow sorting and selection of columns in the Connection Log.

Handle sending the |detach| command when not attached in |-handleInitialResponse:| not |-connectionDidAccept:|.

Update the changelog

Don't use [connection attached] as a condition on which to validate the Stop item.

Add an Attached menu item and give it a keyboard shortcut.

This also binds that item and the attached toolbar item to connection.attached
so both update their UI automatically.

Change the version number to 1.4 Beta 2.

Set the key equivalent for the remove breakpoint button to be the backspace key.

Add a second bug number to the "Stop" button change

Formally define BSSourceViewDelegate as a protocol.

Add drag and drop support on BSSourceView and enable it in the BreakpointController.

Revert "Stub out the methods needed for drag and drop on the breakpoint table view."

This reverts commit d0a336076f5e432b2fe52db4b5f4b10cc37b69cc.

Revert "Add drag and drop for the table view."

This reverts commit 7a8a10f2673ce531fcb38cfec89a04991bbe09c0.

Add drag and drop for the table view.

Stub out the methods needed for drag and drop on the breakpoint table view.

Fix a warning.

Happy new year!

Removed some extra braces and add a comment for NetworkConnection(Private)

Add a Stop menu item.

Split out changes for the different 1.4 betas.

Add a "stop" button to the toolbar that detaches the debugger from the backend for the current session.

Remove two warnings about using deprecated |+[NSString stringWithContentsOfFile:]|.

Fix formatting that Xcode bodged.

Split out the C++ class into its own file. Also move NetworkConnection's private interface out.

Use a C++ class to handle the CFNetwork events, rather than plain C functions.

Forgot to check in modp_b64/. Oops.

Drop in Sparkle 1.5b6 so that it doesn't crash on relaunch anymore.

Don't fetch source in |-[DebuggerBackEnd loadStackFrame:]| for empy filenames.
Doing so wedges the write queue because the error comes back without a transasction ID.

Add a hack to get |-expandVariables| working.

Add a |-description| for VariableNode.

Update changelong for 1.4

Use the unstable AppCast URL.

Renaming because there's too many Debugger* classes
DebuggerProcessor -> DebuggerBackEnd
DebuggerConnection -> NetworkConnection

Move -[DebuggerConnection send:] to the private interface because it isn't threadsafe.

Plug a bunch of leaks by implementing -[StackFrame dealloc]

Merge -[DebuggerController startDebugger] and |-debuggerConnected|.

Update Info.plist for Beta 1.

Add missing |-[super dealloc]| in LoggingController.

Fix a warning about a non-synthesized setter being marked atomic.

Switch out base64 decoders. Also memset the destination string so that there's no garbage data in the result.

Remove the remote host from the title.

Open the help page with HTTPS.

In -[DebuggerController debuggerConnected], do not do anything if we're not attached.

Remove verbose logging from SocketAcceptCallback()

* Store the DebuggerAttached value in the preferences
* Respect the attached checkbox on start

* Remove the AutoReconnect preference item now that reconnecting is a thing of the past
* Remove Icons/Reconnect.png

Do away with the concept of "reconnect" and use a checbox for attached/detached state.

* |-[DebuggerConnectionDelegate connectionDidAccept:]| now actually gets signaled
* When the debugger is in "detached" mode, it will automatically reject new client sockets.

Update the API for loading properties to just pass around VariableNode pointers, rather than string names.

Fix reconnect feature by resetting the last txR and txW values.

Do not ever close the socket until we are definitely closing the connection. We can't reuse the same socket fast enough.

Try to make socket reuse more robust. It isn't working.

Do not ever spawn more than 1 network thread.

Properly quit the run loop. For some reason, calling -[NSRunLoop run] doesn't work with CFRunLoopStop(). Activate it with CFRunLoopRun() instead.

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.