Re: [AD] Empty Datafile Bug

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


> Hm, Evert, Peter: Since many editors seem to do this, and it's often
> hard to find the setting to disable it.. what do you think about one big
> space-removal patch? Should be a matter of a simple grep..

I agree with Peter, now's not the time. If we want to do this, then I'd say
do it for 4.3 once that is changed so drastically that it is basically new
code, if it is nescessary (which it may not even be because much of the
code would be rewritten from scratch anyway). As a proof-of-concept, can
you do a quick check that shows the extend of the problem?
If and when we decide to do this anyway, can we also get rid of the silly
`three space indent, but tab expands to 8 spaces' layout and just write 8
spaces for 8 spaces rather than one tab? My editor can do it just fine, but
it annoys me anyway.

The way I personally deal with those annoying `blank line' changes (if I
spot them) is to make sure my local copy of a file is up to date, edit the
patch (either remove the offending parts, or restore the blanks after the +
sign so that the patch in effect does nothing - whichever is easier in a
particular situation), apply, regenerate and reverse.

Evert




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