[Debian-eeepc-devel] [Debian-eeepc-commits] [SCM] Maintenance of eeepc-acpi-scripts debian package branch, master, updated. 1.1.1-39-g3ab8972

[Ben Armstrong]

On Wed, 14 Oct 2009 10:43:34 +0300
Damyan Ivanov <dmn at debian.org> wrote:

> -=| Darren Salt, Tue, Oct 13, 2009 at 03:17:07PM +0000 |=-
> > The following commit has been merged in the master branch:
> > commit 3ab89721828e085bc6116bd1afe1a4e15833aca3
> > Author: Darren Salt <linux at youmustbejoking.demon.co.uk>
> > Date:   Tue Oct 13 15:59:26 2009 +0100
> > 
> >     Strip out comments from the config file; store a fully-commented version in .../examples/.
> 
> Hm, I find editing commented files much less of a hassle. Havng to 
> open another terminal and look for hints seems like a regression to 
> me. What is the benefit?

Time spent parsing, since we include this everywhere.  According to
Darren's tests:

<_ds_> Hmm. Might be worth making a man page for the defaults
       file and removing the comments from it…
* _ds_ tests and sees a parsing time reduction of about 45%
       (using dash)

Ben
--
 ,-.  nSLUG    http://www.nslug.ns.ca   synrg at sanctuary.nslug.ns.ca
 \`'  Debian   http://www.debian.org    synrg at debian.org
  `          [ gpg 395C F3A4 35D3 D247 1387 2D9E 5A94 F3CA 0B27 13C8 ]