log,core,dnsx: m comments

Author: ignoramous

intra/core/p2est.go

@@ -43,7 +43,7 @@ var _ P2QuantileEstimator = (*p2)(nil)
 
 // NewP50Estimator returns a new P50 (median) estimator.
 func NewP50Estimator(ctx context.Context) *p2 {
-	// calibrate: go.dev/play/p/Ry1i61XqzgB
+	// calibrate: go.dev/play/p/1Gzn6m4GweJ
 	// 31 worked best amid wild latency fluctuations
 	// using 11 for lower overhead; 5 is the default
 	return NewP2QuantileEstimator(ctx, 11, 0.5)

intra/dnsx/transport.go

@@ -1338,6 +1338,7 @@ func (r *resolver) chooseOne(chooseRandom bool, ids ...string) (theone string) {
 	}
 	r.RUnlock()
 
+	// TODO: prefer proxy DNS (wg) when available
 	best, preferred, recoverables, errored, ended := Categorize(trs)
 	if settings.Debug {
 		defer func() {

intra/log/ring.go

@@ -26,6 +26,7 @@ type ring[T any] struct {
 // NewRing creates a new ring buffer with the given capacity (256 if <= 1).
 // evict must be light as it is called for each element displaced by a push, under contention.
 func newRing[T any](ctx context.Context, capacity int, evict func(T)) *ring[T] {
+	// go.dev/play/p/8QUZWqcG5oP
 	if evict == nil {
 		evict = func(T) {}
 	}