[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
signature.asc
Description: This is a digitally signed message part.
- [Bug-wget] Configure cleanup, Darshit Shah, 2014/11/16
- Re: [Bug-wget] Configure cleanup,
Tim Rühsen <=
- Re: [Bug-wget] Configure cleanup, Darshit Shah, 2014/11/16
- Re: [Bug-wget] Configure cleanup, Tim Ruehsen, 2014/11/17
- Re: [Bug-wget] Configure cleanup, Darshit Shah, 2014/11/17
- Re: [Bug-wget] Configure cleanup, Tim Ruehsen, 2014/11/17
- Re: [Bug-wget] Configure cleanup, Darshit Shah, 2014/11/17
- Re: [Bug-wget] Configure cleanup, Tim Ruehsen, 2014/11/18
- Re: [Bug-wget] Configure cleanup, Darshit Shah, 2014/11/18
- Re: [Bug-wget] Configure cleanup, Darshit Shah, 2014/11/20
- Re: [Bug-wget] Configure cleanup, Darshit Shah, 2014/11/20
- Re: [Bug-wget] Configure cleanup, Giuseppe Scrivano, 2014/11/20