bug-wget
[Top][All Lists]
Advanced

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

Re: [Bug-wget] Configure cleanup


From: Tim Rühsen
Subject: Re: [Bug-wget] Configure cleanup
Date: Sun, 16 Nov 2014 15:32:19 +0100
User-agent: KMail/4.14.2 (Linux/3.16.0-4-amd64; KDE/4.14.2; x86_64; ; )

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 !?

Thank you.

Tim

Attachment: signature.asc
Description: This is a digitally signed message part.


reply via email to

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