remove todo's

p2p/session.go

@@ -291,7 +291,7 @@ func (s *session[H]) verify(headers []H) error {
 		if trusted.Height() != s.from.Height() {
 			if trusted.Height()+1 != untrusted.Height() {
 				// Exchange requires requested ranges to always consist of adjacent headers.
-				return fmt.Errorf("peer sent valid??? but non-adjacent header. expected:%d, received:%d",
+				return fmt.Errorf("peer sent valid but non-adjacent header. expected:%d, received:%d",
 					trusted.Height()+1,
 					untrusted.Height(),
 				)

store/batch.go

@@ -84,7 +84,7 @@ func (b *batch[H]) Append(headers ...H) {
 	b.lk.Lock()
 	defer b.lk.Unlock()
 	for _, h := range headers {
-		b.headers = append(b.headers, h) // gap is possible?
+		b.headers = append(b.headers, h)
 		b.heights[h.Hash().String()] = h.Height()
 	}
 }

store/store.go

@@ -395,7 +395,7 @@ func (s *Store[H]) flushLoop() {
 		// and notify waiters if any + increase current read head height
 		// it is important to do Pub after updating pending
 		// so pending is consistent with atomic Height counter on the heightSub
-		s.heightSub.Pub(headers...) // TODO(oleg): the only place where we mutate heightSub.
+		s.heightSub.Pub(headers...)
 		// don't flush and continue if pending batch is not grown enough,
 		// and Store is not stopping(headers == nil)
 		if s.pending.Len() < s.Params.WriteBatchSize && headers != nil {