[Python-Dev] Reindenting unicodedata.c

holger krekel pyth@devel.trillke.net
Fri, 8 Nov 2002 21:52:09 +0100


Guido van Rossum wrote:
> > [Martin v. Loewis]
> > > While working on unicodedata.c, I found that its indentation does not
> > > follow PEP 7. Is it ok to reindent it (i.e. use tabs)?
> > 
> > Probably not, but I'd be in favor of it.  unicodeobject.c too, and _sre.c,
> > and _hotshot.c, and ... the PEP 7 std is simply ignored.
> 
> I'm in favor of reindenting too, but only if you're doing major work
> on a module.  I.e. don't just reindent a module when you're fixing a
> few lines of code; but when you're doing a major refactoring, it
> should be fine.
> 
> CVS-wise, it's better to do the reindent as a separate checkin, so you
> know you're not changing anything else.  (Preferably just *before* you
> start doing any major surgery.)

Probably backporting patches (e.g. python2.2.3) gets harder.

Some bigger projects i know *strictly* prohibit cosmetic-only 
patches because of this very problem.  Only if you are actually
changing code can you reindent.

regards,

    holger