Add a bunch of TODOs

tracker/http.go

@@ -78,7 +78,8 @@ func setAnnounceParams(_url *url.URL, ar *AnnounceRequest) {
 	}
 	// http://stackoverflow.com/questions/17418004/why-does-tracker-server-not-understand-my-request-bittorrent-protocol
 	q.Set("compact", "1")
-	// According to https://wiki.vuze.com/w/Message_Stream_Encryption.
+	// According to https://wiki.vuze.com/w/Message_Stream_Encryption. TODO:
+	// Take EncryptionPolicy or something like it as a parameter.
 	q.Set("supportcrypto", "1")
 
 	_url.RawQuery = q.Encode()

tracker/tracker.go

@@ -47,6 +47,8 @@ var (
 	ErrBadScheme = errors.New("unknown scheme")
 )
 
+// TODO: Just split udp/http announcing completely, to support various different options they have.
+
 func Announce(cl *http.Client, userAgent string, urlStr string, req *AnnounceRequest) (res AnnounceResponse, err error) {
 	return AnnounceHost(cl, userAgent, urlStr, req, "")
 }

tracker/udp.go

@@ -257,6 +257,8 @@ func (c *udpAnnounce) connect() (err error) {
 	return
 }
 
+// TODO: Split on IPv6, as BEP 15 says response peer decoding depends on
+// network in use.
 func announceUDP(ar *AnnounceRequest, _url *url.URL) (AnnounceResponse, error) {
 	ua := udpAnnounce{
 		url: *_url,