[devel] [PATCH for apt 1/2] Add option for debugging DynamicMMap::Allocate

Dmitry V. Levin ldv на altlinux.org
Пн Дек 9 13:24:37 MSK 2019


On Mon, Dec 09, 2019 at 09:58:55AM +0300, Aleksei Nikiforov wrote:
> 09.12.2019 1:50, Dmitry V. Levin пишет:
> > On Fri, Dec 06, 2019 at 04:16:05PM +0300, Aleksei Nikiforov wrote:
> >> ---
> >>   apt/apt-pkg/contrib/mmap.cc | 15 +++++++++++++++
> >>   apt/doc/apt.conf.5.sgml     |  2 +-
> >>   2 files changed, 16 insertions(+), 1 deletion(-)
> > 
> > I suggest adding the name of the new option to the commit message
> > so it would be git-grep'able, for example:
> > 
> > Add Debug::DynamicMMap::Allocate option
> > 
> > Add a new option for debugging DynamicMMap::Allocate.
> 
> But option name is already in commit message. See the email subject 
> which is also a part of commit message. Why is it needed to duplicate it?

I don't see a duplication here, but anyway, this is just an example, and
"Add Debug::DynamicMMap::Allocate option" would be better than
"Add option for debugging DynamicMMap::Allocate".


-- 
ldv
----------- следующая часть -----------
Было удалено вложение не в текстовом формате...
Имя     : signature.asc
Тип     : application/pgp-signature
Размер  : 801 байтов
Описание: отсутствует
Url     : <http://lists.altlinux.org/pipermail/devel/attachments/20191209/b55dfb7e/attachment.bin>


Подробная информация о списке рассылки Devel