Re: [PATCH v3 11/16] refs.c: kill register_ref_store(), add register_submodule_ref_store()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH v3 11/16] refs.c: kill register_ref_store(), add register_submodule_ref_store()
From: Junio C Hamano <gitster@xxxxxxxxx>
Date: Fri, 17 Feb 2017 11:29:31 -0800
Cc: git@xxxxxxxxxxxxxxx, Michael Haggerty <mhagger@xxxxxxxxxxxx>, Johannes Schindelin <Johannes.Schindelin@xxxxxx>, Ramsay Jones <ramsay@xxxxxxxxxxxxxxxxxxxx>, Stefan Beller <sbeller@xxxxxxxxxx>, novalis@xxxxxxxxxxx
In-reply-to: <20170217140436.17336-12-pclouds@gmail.com> ("Nguyễn Thái Ngọc Duy"'s message of "Fri, 17 Feb 2017 21:04:31 +0700")
References: <20170216114818.6080-1-pclouds@gmail.com> <20170217140436.17336-1-pclouds@gmail.com> <20170217140436.17336-12-pclouds@gmail.com>
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1.91 (gnu/linux)

Nguyễn Thái Ngọc Duy  <pclouds@xxxxxxxxx> writes:

> This is the last function in this code (besides public API) that takes
> submodule argument and handles both main/submodule cases. Break it down,
> move main store registration in get_main_ref_store() and keep the rest
> in register_submodule_ref_store().

Very nice.

References:
- [PATCH v2 00/16] Remove submodule from files-backend.c
  - From: Nguyễn Thái Ngọc Duy
- [PATCH v3 00/16] Remove submodule from files-backend.c
  - From: Nguyễn Thái Ngọc Duy
- [PATCH v3 11/16] refs.c: kill register_ref_store(), add register_submodule_ref_store()
  - From: Nguyễn Thái Ngọc Duy

Prev by Date: Re: [PATCH v3 04/16] files-backend: replace *git_path*() with files_path()
Next by Date: Re: [PATCH 3/3] rename_ref: replace empty deletion message in HEAD's log
Previous by thread: [PATCH v3 11/16] refs.c: kill register_ref_store(), add register_submodule_ref_store()
Next by thread: [PATCH v3 12/16] refs.c: make get_main_ref_store() public and use it
Index(es):
- Date
- Thread

[Index of Archives] [Linux Kernel Development] [Gcc Help] [IETF Annouce] [DCCP] [Netdev] [Networking] [Security] [V4L] [Bugtraq] [Yosemite] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux SCSI] [Fedora Users]