GNU bug report logs - #31687
[PATCH] services: Add dnsmasq-service-type.

Previous Next

Full log

Message #16 received at 31687 <at> debbugs.gnu.org (full text, mbox):

From: iyzsong <at> member.fsf.org (宋文武)
To: ludo <at> gnu.org (Ludovic Courtès)
Cc: 31687 <at> debbugs.gnu.org
Subject: Re: [bug#31687] [PATCH] services: Add dnsmasq-service-type.
Date: Fri, 08 Jun 2018 23:45:26 +0800

ludo <at> gnu.org (Ludovic Courtès) writes:

> Hello!
>
> Thanks for this patch, that looks like a useful service to have!
>
> 宋文武 <iyzsong <at> member.fsf.org> skribis:
>
>> +@item @code{no-negcache?} (default: @code{#f})
>> +When true, disable negative caching.
>
> Minor issues:
>
>   • The general guideline is to avoid abbreviations in identifiers—we’re
>     using the language that comes with ‘call-with-current-continuation’,
>     after all.  ;-)
>
>   • Another guideline is to avoid double-negations and the likes.
>
> So in this example, I would recommend:
>
>   ‘negative-caching?’ (default: #t)
>

Agree, I like this naming guideline, pushed :-)


> I think there’s a couple of other abbreviations.

It’s not clear to me whether other options are abbreviations or not or
worth to expand.  eg: ‘no-hosts?’ and ‘no-resolv?’, they’re refer to the
well-known ‘/etc/hosts’ and ‘/etc/resolv.conf’ files, what could we use
instead?

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.