Update transport lifetime documentation The original was written before any code was written and had nothing to do with the way things are actually done. Signed-off-by: Carlos Martín Nieto <carlos@cmartin.tk>
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41
diff --git a/src/transport.h b/src/transport.h
index e809734..b14684b 100644
--- a/src/transport.h
+++ b/src/transport.h
@@ -26,11 +26,16 @@ typedef struct git_transport_caps {
* making use of the direction if necessary. This function must also
* store the remote heads and any other information it needs.
*
- * If we just want to execute ls-remote, ->ls() gets
- * called. Otherwise, the have/want/need list needs to be built via
- * ->wanthaveneed(). We can then ->push() or ->pull(). When we're
- * done, we call ->close() to close the connection. ->free() takes
- * care of freeing all the resources.
+ * The next useful step is to call ->ls() to get the list of
+ * references available to the remote. These references may have been
+ * collected on connect, or we may build them now. For ls-remote,
+ * nothing else is needed other than closing the connection.
+ * Otherwise, the higher leves decide which objects we want to
+ * have. ->send_have() is used to tell the other end what we have. If
+ * we do need to download a pack, ->download_pack() is called.
+ *
+ * When we're done, we call ->close() to close the
+ * connection. ->free() takes care of freeing all the resources.
*/
struct git_transport {
@@ -52,14 +57,6 @@ struct git_transport {
*/
int (*ls)(struct git_transport *transport, git_headarray *headarray);
/**
- * Calculate want/have/need. May not even be needed.
- */
- int (*wanthaveneed)(struct git_transport *transport, void *something);
- /**
- * Build the pack
- */
- int (*build_pack)(struct git_transport *transport);
- /**
* Push the changes over
*/
int (*push)(struct git_transport *transport);