From bbda440f15080ce9c46a64abba92c9e790959334 Mon Sep 17 00:00:00 2001
From: Vincent Ambo <tazjin@google.com>
Date: Mon, 28 Oct 2019 22:39:12 +0100
Subject: [PATCH] chore(server): Remove outdated TODO

Real-life experience has shown that the weighting of the metric
produced here is appropriate.
---
 server/layers/grouping.go | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/server/layers/grouping.go b/server/layers/grouping.go
index 1fbbf33..74952a1 100644
--- a/server/layers/grouping.go
+++ b/server/layers/grouping.go
@@ -300,11 +300,7 @@ func groupLayer(dt *flow.DominatorTree, root *closure) Layer {
 	sort.Strings(contents)
 
 	return Layer{
-		Contents: contents,
-		// TODO(tazjin): The point of this is to factor in
-		// both the size and the popularity when making merge
-		// decisions, but there might be a smarter way to do
-		// it than a plain multiplication.
+		Contents:    contents,
 		MergeRating: uint64(root.Popularity) * size,
 	}
 }