[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug-wget] Configure cleanup

From: Darshit Shah
Subject: Re: [Bug-wget] Configure cleanup
Date: Mon, 17 Nov 2014 09:48:30 +0530
User-agent: Mutt/1.5.23 (2014-03-12)

On 11/16, Tim Rühsen wrote:
Am Sonntag, 16. November 2014, 16:59:18 schrieb Darshit Shah:
I cleanup up configure.ac slightly.

It's not complete, but this is in my opinion a step up from what we had
already. If everyone is fine with it, I'll push this version of the patch.

The first patch adds the -Wextra CFLAG by default on systems where a CFLAG
variable is not already defined.

It works for me, just a comment.

configure.ac seems to have 2 spaces indentation - now we have 4 spaces mixed
with 2 spaces indentation. At some places the indentation is wrong.
I personally prefer 2 spaces - but it really doesn't matter. Maybe you amend
configure.ac so that we have the same indentation everywhere !?

You're right. The indentation and formatting is quite a bit off. In fact, that is one reason which is preventing any further work on the configure.ac file for me. I'm stuck and unable to understand how that code ought to be formatted. I'll look into it once again and see if I can finally format it with two spaces and other stylistic guidelines.

Thanking You,
Darshit Shah

Attachment: pgpxs8YIvIaHb.pgp
Description: PGP signature

reply via email to

[Prev in Thread] Current Thread [Next in Thread]