I don't believe that tcc has a whitespace policy (though it appears that
nearly all indentation is with spaces in the C sources, and every once in a
while a git mixed tabs+spaces where the tabs look like they are supposed to
be eight characters wide). As such, I would suggest minimizing the number
of patches with whitespace changes. If you are going to cleanup whitespace
(and your fourth patch makes it clear this is on your mind), I think all
whitespace should be cleaned up in all files, in one go.

The cleanups you have look fine by me, for what it's worth.

David

On Wed, Nov 26, 2014 at 3:38 AM, Lee Duhem <[email protected]> wrote:

> Hi,
>
> Some adjustments miss in the previous patch,
> here is the updated version.
>
> Sincerely,
> lee
>
>
> On Wed, Nov 26, 2014 at 4:01 PM, Lee Duhem <[email protected]> wrote:
> > Hi,
> >
> > This basically a code style improvement patch. Changes included
> > in this patch:
> >
> > 1. adjust indentations;
> > 2. make `configure --help` return success instead of failure;
> > 3. remove some tailing spaces.
> >
> > Sincerely,
> > lee
>
> _______________________________________________
> Tinycc-devel mailing list
> [email protected]
> https://lists.nongnu.org/mailman/listinfo/tinycc-devel
>
>


-- 
 "Debugging is twice as hard as writing the code in the first place.
  Therefore, if you write the code as cleverly as possible, you are,
  by definition, not smart enough to debug it." -- Brian Kernighan
_______________________________________________
Tinycc-devel mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/tinycc-devel

Reply via email to