Updated godoc

Author: durkmurder

engine/common/follower.go

@@ -17,10 +17,10 @@ type FollowerCore interface {
 	// The originID parameter identifies the node that sent the batch of blocks.
 	// The connectedRange parameter contains the blocks, they must form a sequence of connected blocks.
 	// No errors are expected during normal operations.
-	// This function is safe to use in concurrent environment.
+	// Implementors need to ensure that this function is safe to be used in concurrent environment.
 	OnBlockRange(originID flow.Identifier, connectedRange []*flow.Block) error
 	// OnFinalizedBlock is called when a new block is finalized by Hotstuff.
 	// FollowerCore updates can update its local state using this information.
-	// This function is safe to use in concurrent environment.
+	// Implementors need to ensure that this function is safe to be used in concurrent environment.
 	OnFinalizedBlock(finalized *flow.Header)
 }

engine/common/follower/core.go

@@ -216,9 +216,9 @@ func (c *Core) processCoreSeqEvents(ctx irrecoverable.SignalerContext, ready com
 	}
 }
 
-// OnFinalizedBlock updates local state of pendingCache tree using received finalized block.
-// Is NOT concurrency safe, has to be used by the same goroutine as extendCertifiedBlocks.
-// OnFinalizedBlock and extendCertifiedBlocks MUST be sequentially ordered.
+// OnFinalizedBlock updates local state of pendingCache tree using received finalized block and queues finalized block
+// to be processed by internal goroutine.
+// This function is safe to use in concurrent environment.
 func (c *Core) OnFinalizedBlock(final *flow.Header) {
 	c.pendingCache.PruneUpToView(final.View)