diff --git a/addrmgr/addrmanager.go b/addrmgr/addrmanager.go index 16c66f71..b1c32b5e 100644 --- a/addrmgr/addrmanager.go +++ b/addrmgr/addrmanager.go @@ -169,7 +169,7 @@ func (a *AddrManager) updateAddress(netAddr, srcAddr *wire.NetAddress) { addr := NetAddressKey(netAddr) ka := a.find(netAddr) if ka != nil { - // TODO(oga) only update addresses periodically. + // TODO: only update addresses periodically. // Update the last seen time and services. // note that to prevent causing excess garbage on getaddr // messages the netaddresses in addrmaanger are *immutable*, diff --git a/blockmanager.go b/blockmanager.go index c85e64d8..10d2cfc0 100644 --- a/blockmanager.go +++ b/blockmanager.go @@ -364,7 +364,7 @@ func (b *blockManager) handleDonePeerMsg(peers *list.List, sp *serverPeer) { // Remove requested blocks from the global map so that they will be // fetched from elsewhere next time we get an inv. - // TODO(oga) we could possibly here check which peers have these blocks + // TODO: we could possibly here check which peers have these blocks // and request them now to speed things up a little. for k := range sp.requestedBlocks { delete(b.requestedBlocks, k) diff --git a/btcd.go b/btcd.go index 5c68dc19..88921192 100644 --- a/btcd.go +++ b/btcd.go @@ -126,7 +126,7 @@ func btcdMain(serverChan chan<- *server) error { // Create server and start it. server, err := newServer(cfg.Listeners, db, activeNetParams.Params) if err != nil { - // TODO(oga) this logging could do with some beautifying. + // TODO: this logging could do with some beautifying. btcdLog.Errorf("Unable to start server on %v: %v", cfg.Listeners, err) return err diff --git a/btcec/pubkey.go b/btcec/pubkey.go index be6f8509..f145414e 100644 --- a/btcec/pubkey.go +++ b/btcec/pubkey.go @@ -25,7 +25,7 @@ func isOdd(a *big.Int) bool { // decompressPoint decompresses a point on the given curve given the X point and // the solution to use. func decompressPoint(curve *KoblitzCurve, x *big.Int, ybit bool) (*big.Int, error) { - // TODO(oga) This will probably only work for secp256k1 due to + // TODO: This will probably only work for secp256k1 due to // optimizations. // Y = +-sqrt(x^3 + B) diff --git a/btcec/signature.go b/btcec/signature.go index ab5a345c..ae94e81b 100644 --- a/btcec/signature.go +++ b/btcec/signature.go @@ -329,7 +329,7 @@ func recoverKeyFromSignature(curve *KoblitzCurve, sig *Signature, msg []byte, e.Mod(e, curve.Params().N) minuseGx, minuseGy := curve.ScalarBaseMult(e.Bytes()) - // TODO(oga) this would be faster if we did a mult and add in one + // TODO: this would be faster if we did a mult and add in one // step to prevent the jacobian conversion back and forth. Qx, Qy := curve.Add(sRx, sRy, minuseGx, minuseGy) diff --git a/rpcserver.go b/rpcserver.go index 4f07f487..972176be 100644 --- a/rpcserver.go +++ b/rpcserver.go @@ -4221,7 +4221,7 @@ func newRPCServer(listenAddrs []string, generator *mining.BlkTmplGenerator, s *s } } - // TODO(oga) this code is similar to that in server, should be + // TODO: this code is similar to that in server, should be // factored into something shared. ipv4ListenAddrs, ipv6ListenAddrs, _, err := parseListeners(listenAddrs) if err != nil { diff --git a/server.go b/server.go index 668dc991..5897172f 100644 --- a/server.go +++ b/server.go @@ -1227,7 +1227,7 @@ func (s *server) handleAddPeerMsg(state *peerState, sp *serverPeer) bool { srvrLog.Infof("Max peers reached [%d] - disconnecting peer %s", cfg.MaxPeers, sp) sp.Disconnect() - // TODO(oga) how to handle permanent peers here? + // TODO: how to handle permanent peers here? // they should be rescheduled. return false } @@ -1439,7 +1439,7 @@ func (s *server) handleQuery(state *peerState, querymsg interface{}) { msg.reply <- peers case connectNodeMsg: - // XXX(oga) duplicate oneshots? + // TODO: duplicate oneshots? // Limit max number of total peers. if state.Count() >= cfg.MaxPeers { msg.reply <- errors.New("max peers reached") @@ -1462,7 +1462,7 @@ func (s *server) handleQuery(state *peerState, querymsg interface{}) { return } - // TODO(oga) if too many, nuke a non-perm peer. + // TODO: if too many, nuke a non-perm peer. go s.connManager.Connect(&connmgr.ConnReq{ Addr: netAddr, Permanent: msg.permanent, @@ -2119,9 +2119,9 @@ out: for { select { case <-timer.C: - // TODO(oga) pick external port more cleverly - // TODO(oga) know which ports we are listening to on an external net. - // TODO(oga) if specific listen port doesn't work then ask for wildcard + // TODO: pick external port more cleverly + // TODO: know which ports we are listening to on an external net. + // TODO: if specific listen port doesn't work then ask for wildcard // listen port? // XXX this assumes timeout is in seconds. listenPort, err := s.nat.AddPortMapping("tcp", int(lport), int(lport), @@ -2130,7 +2130,7 @@ out: srvrLog.Warnf("can't add UPnP port mapping: %v", err) } if first && err == nil { - // TODO(oga): look this up periodically to see if upnp domain changed + // TODO: look this up periodically to see if upnp domain changed // and so did ip. externalip, err := s.nat.GetExternalAddress() if err != nil { @@ -2229,7 +2229,7 @@ func newServer(listenAddrs []string, db database.DB, chainParams *chaincfg.Param // nil nat here is fine, just means no upnp on network. } - // TODO(oga) nonstandard port... + // TODO: nonstandard port... if wildcard { port, err := strconv.ParseUint(activeNetParams.DefaultPort, diff --git a/txscript/sign.go b/txscript/sign.go index f50b692a..8960e6b0 100644 --- a/txscript/sign.go +++ b/txscript/sign.go @@ -168,7 +168,7 @@ func mergeScripts(chainParams *chaincfg.Params, tx *wire.MsgTx, idx int, pkScript []byte, class ScriptClass, addresses []btcutil.Address, nRequired int, sigScript, prevScript []byte) []byte { - // TODO(oga) the scripthash and multisig paths here are overly + // TODO: the scripthash and multisig paths here are overly // inefficient in that they will recompute already known data. // some internal refactoring could probably make this avoid needless // extra calculations.