Style strangenesses (was: cvs commit: src/etc rc.subr)

[Ruslan Ermilov]

On Tue, Nov 28, 2006 at 09:06:44AM +1030, Greg 'groggy' Lehey wrote:
> On Monday, 27 November 2006 at 16:03:24 +0300, Yar Tikhiy wrote:
> > By the way, now the output follows the NetBSD rc.conf style:
> >
> > # $foo_enable
> > foo_enable=YES
> >
> > Could it be changed even further to match our own rc.conf style?
> > I.e.:
> >
> > # $foo_enable
> > foo_enable="YES"
> 
> Can somebody justify this style to me?  It seems unnecessarily
> confusing, like Microsoft mail headers with
> 
>   To: "'Fred Bloggs'" <fred at nowhere.com>
> 
> I'm left with the feeling of sloppiness.
> 
Quotes are needed if a value has spaces.  Quotes around values
without spaces aren't necessary but nice for uniformity.

That said, I don't normally use quotes in /boot/loader.conf.


Cheers,
-- 
Ruslan Ermilov
ru at FreeBSD.org
FreeBSD committer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-src/attachments/20061128/5e24a16c/attachment.pgp