| author | Chris Jones <christian.jones@sri.com> |
| Sun, 09 Feb 2014 09:50:38 -0700 | |
| changeset 182 | 626c390682fc |
| parent 156 | 520ccb3e05d4 |
| child 157 | eadf15a06ff5 |
| child 183 | b4bd77d58a3e |
| permissions | -rw-r--r-- |
|
150
fa7f6ff10c67
Code reorder and doc cleanup.
Chris Jones <christian.jones@sri.com>
parents:
146
diff
changeset
|
1 |
Review all the *Client receiver methods in layer3.go. They should |
|
fa7f6ff10c67
Code reorder and doc cleanup.
Chris Jones <christian.jones@sri.com>
parents:
146
diff
changeset
|
2 |
probably either all be receivers, or none. |
| 119 | 3 |
|
4 |
Add a Reconnect() function. |
|
5 |
||
6 |
Eliminate as many uses of Generic as possible. |
|
| 152 | 7 |
|
|
153
bbd4166df95d
Simplified the API: There's only one constructor, and it does everything necessary to initiate the stream. StartSession() and Roster.Update() have both been eliminated.
Chris Jones <christian.jones@sri.com>
parents:
152
diff
changeset
|
8 |
Don't keep the password in memory once we're done with it. |
|
bbd4166df95d
Simplified the API: There's only one constructor, and it does everything necessary to initiate the stream. StartSession() and Roster.Update() have both been eliminated.
Chris Jones <christian.jones@sri.com>
parents:
152
diff
changeset
|
9 |
|
|
bbd4166df95d
Simplified the API: There's only one constructor, and it does everything necessary to initiate the stream. StartSession() and Roster.Update() have both been eliminated.
Chris Jones <christian.jones@sri.com>
parents:
152
diff
changeset
|
10 |
Rename extension.StanzaHandlers to something like StanzaTypes. |
|
bbd4166df95d
Simplified the API: There's only one constructor, and it does everything necessary to initiate the stream. StartSession() and Roster.Update() have both been eliminated.
Chris Jones <christian.jones@sri.com>
parents:
152
diff
changeset
|
11 |
|
|
bbd4166df95d
Simplified the API: There's only one constructor, and it does everything necessary to initiate the stream. StartSession() and Roster.Update() have both been eliminated.
Chris Jones <christian.jones@sri.com>
parents:
152
diff
changeset
|
12 |
Think about how to gracefully shutdown. Probably have a Close() |
|
bbd4166df95d
Simplified the API: There's only one constructor, and it does everything necessary to initiate the stream. StartSession() and Roster.Update() have both been eliminated.
Chris Jones <christian.jones@sri.com>
parents:
152
diff
changeset
|
13 |
function. |
| 154 | 14 |
|
15 |
Get rid of logging. We're providing status updates. Allow some sort of |
|
16 |
debug flag that prints from layer1. |