From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?q?Nguy=E1=BB=85n=20Th=C3=A1i=20Ng=E1=BB=8Dc=20Duy?= Subject: [PATCH v2 03/27] upload-pack: move shallow deepen code out of receive_needs() Date: Sun, 12 Jun 2016 17:53:45 +0700 Message-ID: <20160612105409.22156-4-pclouds@gmail.com> References: <20160610122714.3341-1-pclouds@gmail.com> <20160612105409.22156-1-pclouds@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Junio C Hamano , Eric Sunshine , =?UTF-8?q?Nguy=E1=BB=85n=20Th=C3=A1i=20Ng=E1=BB=8Dc=20Duy?= To: git@vger.kernel.org X-From: git-owner@vger.kernel.org Sun Jun 12 12:55:10 2016 Return-path: Envelope-to: gcvg-git-2@plane.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1bC32q-00032V-U0 for gcvg-git-2@plane.gmane.org; Sun, 12 Jun 2016 12:55:09 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752581AbcFLKzE convert rfc822-to-quoted-printable (ORCPT ); Sun, 12 Jun 2016 06:55:04 -0400 Received: from mail-pa0-f65.google.com ([209.85.220.65]:36304 "EHLO mail-pa0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751117AbcFLKzC (ORCPT ); Sun, 12 Jun 2016 06:55:02 -0400 Received: by mail-pa0-f65.google.com with SMTP id fg1so8409022pad.3 for ; Sun, 12 Jun 2016 03:55:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=sovsKiQOiCn9/RBO76HCyqQHgctyGic1choKRUle/rE=; b=xp0+KN2ln+I22dXY5CdHHqx8ZqWt5aNIRoyZpuji7WbQXlpqAH54BNx+VB0PMk+qSc 00arwI2RznbLAcz3omG7W6qoZdr0kJXQlhGm9BT1Vu0or962HGoORo+Akleby7lwTiv1 Z06+PriK54LsaU4gn2AfEcHyV/Q3J6BGcyoWuQhoEg2UB0z0ZdPWJIOjjuaYaWsXCMCn zc+1R7dF7mhM05ZHAn35wGe6Ch6qxIJ6xpJGa5r/uJTmJk9JQTKfLcRBQoFfO6RAbqTM KnDe4pyVByP4PYg9TUKQB8y5q/7LcVtxne2WuD0yWNbMGaIGQKzjX8Cbc9tuqXMmxMEp xQMQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=sovsKiQOiCn9/RBO76HCyqQHgctyGic1choKRUle/rE=; b=Vao21Jl0Vnc72zodTuc+ScW6vw2Ocgdc+UstQb9msbJyzvYWBonj+QS8hABpg+hBE9 SIdyAM2Eb5Ptk1zHeZAoNpqvfXohWoHb1Db7HdX0K3d3gKEYpGbmEBdDkijZ7o64gFsn Kt3AhWWtFQxmDLfcgqYRRVxHJpPNnbBdGy8uAr2j6KIXuOaafrnNxGuIefbkYSamL3yT X4FjMTNRsnY3BwFDw7iS5Ot3yJs3PW3Ih/wTBWaYfG4XreNXCj8bwxo8ShobnGk7sV+7 ZYecijVBB4O01bRmEgI8zJKpgM9JCiRWP/l5eZhWkPqGDFfsKN303wppvx2x1MTfyEmx pD5A== X-Gm-Message-State: ALyK8tIHWbp3dptsp9c8z9xACacdiDHDlpG6eTycBXdXYcLB6tRWdq3L0CNBfnbaJBmd5g== X-Received: by 10.66.141.76 with SMTP id rm12mr14364538pab.129.1465728901299; Sun, 12 Jun 2016 03:55:01 -0700 (PDT) Received: from ash ([115.76.211.1]) by smtp.gmail.com with ESMTPSA id p1sm29819566paz.8.2016.06.12.03.54.58 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sun, 12 Jun 2016 03:55:00 -0700 (PDT) Received: by ash (sSMTP sendmail emulation); Sun, 12 Jun 2016 17:54:56 +0700 X-Mailer: git-send-email 2.8.2.524.g6ff3d78 In-Reply-To: <20160612105409.22156-1-pclouds@gmail.com> Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: This is a prep step for further refactoring. Besides reindentation and s/shallows\./shallows->/g, no other changes are expected. Signed-off-by: Nguy=E1=BB=85n Th=C3=A1i Ng=E1=BB=8Dc Duy Signed-off-by: Junio C Hamano --- upload-pack.c | 99 +++++++++++++++++++++++++++++++--------------------= -------- 1 file changed, 52 insertions(+), 47 deletions(-) diff --git a/upload-pack.c b/upload-pack.c index b3f6653..97ed620 100644 --- a/upload-pack.c +++ b/upload-pack.c @@ -538,6 +538,55 @@ error: } } =20 +static void deepen(int depth, const struct object_array *shallows) +{ + struct commit_list *result =3D NULL, *backup =3D NULL; + int i; + if (depth =3D=3D INFINITE_DEPTH && !is_repository_shallow()) + for (i =3D 0; i < shallows->nr; i++) { + struct object *object =3D shallows->objects[i].item; + object->flags |=3D NOT_SHALLOW; + } + else + backup =3D result =3D + get_shallow_commits(&want_obj, depth, + SHALLOW, NOT_SHALLOW); + while (result) { + struct object *object =3D &result->item->object; + if (!(object->flags & (CLIENT_SHALLOW|NOT_SHALLOW))) { + packet_write(1, "shallow %s", + oid_to_hex(&object->oid)); + register_shallow(object->oid.hash); + shallow_nr++; + } + result =3D result->next; + } + free_commit_list(backup); + for (i =3D 0; i < shallows->nr; i++) { + struct object *object =3D shallows->objects[i].item; + if (object->flags & NOT_SHALLOW) { + struct commit_list *parents; + packet_write(1, "unshallow %s", + oid_to_hex(&object->oid)); + object->flags &=3D ~CLIENT_SHALLOW; + /* make sure the real parents are parsed */ + unregister_shallow(object->oid.hash); + object->parsed =3D 0; + parse_commit_or_die((struct commit *)object); + parents =3D ((struct commit *)object)->parents; + while (parents) { + add_object_array(&parents->item->object, + NULL, &want_obj); + parents =3D parents->next; + } + add_object_array(object, NULL, &extra_edge_obj); + } + /* make sure commit traversal conforms to client */ + register_shallow(object->oid.hash); + } + packet_flush(1); +} + static void receive_needs(void) { struct object_array shallows =3D OBJECT_ARRAY_INIT; @@ -630,53 +679,9 @@ static void receive_needs(void) =20 if (depth =3D=3D 0 && shallows.nr =3D=3D 0) return; - if (depth > 0) { - struct commit_list *result =3D NULL, *backup =3D NULL; - int i; - if (depth =3D=3D INFINITE_DEPTH && !is_repository_shallow()) - for (i =3D 0; i < shallows.nr; i++) { - struct object *object =3D shallows.objects[i].item; - object->flags |=3D NOT_SHALLOW; - } - else - backup =3D result =3D - get_shallow_commits(&want_obj, depth, - SHALLOW, NOT_SHALLOW); - while (result) { - struct object *object =3D &result->item->object; - if (!(object->flags & (CLIENT_SHALLOW|NOT_SHALLOW))) { - packet_write(1, "shallow %s", - oid_to_hex(&object->oid)); - register_shallow(object->oid.hash); - shallow_nr++; - } - result =3D result->next; - } - free_commit_list(backup); - for (i =3D 0; i < shallows.nr; i++) { - struct object *object =3D shallows.objects[i].item; - if (object->flags & NOT_SHALLOW) { - struct commit_list *parents; - packet_write(1, "unshallow %s", - oid_to_hex(&object->oid)); - object->flags &=3D ~CLIENT_SHALLOW; - /* make sure the real parents are parsed */ - unregister_shallow(object->oid.hash); - object->parsed =3D 0; - parse_commit_or_die((struct commit *)object); - parents =3D ((struct commit *)object)->parents; - while (parents) { - add_object_array(&parents->item->object, - NULL, &want_obj); - parents =3D parents->next; - } - add_object_array(object, NULL, &extra_edge_obj); - } - /* make sure commit traversal conforms to client */ - register_shallow(object->oid.hash); - } - packet_flush(1); - } else + if (depth > 0) + deepen(depth, &shallows); + else if (shallows.nr > 0) { int i; for (i =3D 0; i < shallows.nr; i++) --=20 2.8.2.524.g6ff3d78