Re: [AD] Re: CVS: allegro indent.pro,1.1.1.1,1.2

[ Thread Index | Date Index | More lists.liballeg.org/allegro-developers Archives ]


On Sun, 2005-08-21 at 23:17 +0200, Evert Glebbeek wrote:
> On Sunday 21 August 2005 23:13, Elias Pschernig wrote:
> > --kr -nce -ss -ncs -i3 -cli3
> > +-kr -nce -ss -ncs -i3 -cli3 -nut
> 
> Are you sure this is a good idea? We don't want people editing a sourcefile 
> and then running indent over it before diffing it. That way we would end 
> up with a load of tab->space conversions that are not relevant to the 
> patch and that we don't want to apply along with it.
> Maybe an academic situation, but best to avoid it I think.
> 

My impression was, the indent.pro is only useful for new files, since on
existing files it wraps around long lines and destroys things like
DIALOG arrays.. so it should be ok.

-- 
Elias Pschernig





Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/