Re: [PATCHES] various commenting, whitespace changes

[Andy Wingo <wingo@pobox.com>]

Hi Thien-Thi,

Let me say first that I am really happy to have your energetic,
experienced hand going over Guile sources. Thank you! I do think that
small things are important, and your willingness to look at the small
*and* the big is much appreciated. It's nice to have thoughtful, local
commits.

On Fri 27 Aug 2010 01:29, Thien-Thi Nguyen <ttn@gnuvola.org> writes:

> I will avoid general cleanup, but i don't intend to avoid cleanup on
> files i wish to hack on (for now, this set includes makefiles, build
> scripts and documentation).

Fair enough. Please continue to separate cleanup commits from other
commits.

> I sidestepped [issues in guile-func-name-check], by rewriting in
> Scheme (and using Emacs' ‘indent-sexp’).  Unfortunately, it looks like
> hydra is not liking it:
>
> http://hydra.nixos.org/build/606792
>
> Perhaps a workaround would be to use ‘--no-autocompile’ in the
> invocation command (libguile/guile-snarf-docs.in:55), but that seems
> ugly and throws away a bug-{hunt,squash}ing opportunity.  WDYT?

I think that not autocompiling is the right thing in this case. We don't
actually want to autocompile in Guile when, um, compiling :)

Also in the future, please run clean builds when you change invocations
of guile within libguile/.  Thanks :)

> Another related idea: Since Guile supports many languages, why not AWK?

That would be fun!

Cheers,

Andy
-- 
http://wingolog.org/