Re: [Xen-devel] [PATCH] replace tabs with spaces of Python code

[Christian Limpach <christian.limpach@xxxxxxxxx>]

On 5/19/05, aq <aquynh@xxxxxxxxx> wrote:
> > The script from python.org that Anthony posted seems to fit the bill.
> 
> no, that script doesnt work very well, because:
> 1. almost all the code indented by 4 spaces, but his script convert
> tab to 8 spaces
> 2. some code consider tab as 4 and 8 spaces in the same file, so it is
> hard to use this script.
> 3. that script doesnt see how python deal with the code. for example,
> lines the python code is aligned with the previous line, but the
> script doesnt take that into account. that is a problem.
> 
> a small example: file tools/python/xen/xm/main.py has few tabs inside,
> and here is how Anthony's script converts the code, and it does it
> wrongly. i have 3 files to compare here, so you can make a conclusion:
> the main.org.py (copy from original main.py), main-t2s.py (converted
> with Anthony's script, with tabsize=4) and main-aq.py (my script,
> coverted manually)
> 
> now to see the problem, here is a hunk taken from the diffs. see the
> commented i put next to the broken line:

That's because you broke the script by changing tabsize to 4.  As
Anthony wrote, the python manual describes the algorithm used to
expand tabs and how tabs get expanded has nothing to with how you have
indented your program.  Whether your indention levels are 4 character
wide or 8 doesn't matter, a tab always means go to the next 8
character column.  The original main.py looks visually incorrect at
the args.append line (it's indented by 8 characters instead of 4) but
that's not a bug.

> > I agree that getting rid of the tabs is important, but it can happen
> > after some of the outstanding tools merges get completed.
> 
> i think it is better to do it now, when there are not much code having
> problem. in the future, code get bigger, so it is more difficult to
> convert and check everything.

I'm not too keen on seeing this done in any of the 2.0 trees.

    christian

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel