Why didn't we mark want_obj as ~UNINTERESTING in the old code?

Is this something we would want to do regardless of shallow clone?

upload-pack.c

@@ -138,6 +138,7 @@ static void create_pack_file(void)
 		} else {
 			for (i = 0; i < want_obj.nr; i++) {
 				struct object *o = want_obj.objects[i].item;
+				/* why??? */
 				o->flags &= ~UNINTERESTING;
 				add_pending_object(&revs, o, NULL);
 			}