[MPlayer-users] Patches for changing indentation accepted?
Fredrik Kuivinen
freku045 at student.liu.se
Sun May 5 15:12:01 CEST 2002
On Sun, May 05, 2002 at 09:30:01AM -0300, Rogério Brito wrote:
> [Automatic answer: RTFM (read DOCS, FAQ), also read DOCS/bugreports.html]
> On May 05 2002, Arpi wrote:
> > no
> > the only case when it's acce[ted, is when the maintainer/author of a
> > code part changed indent in his own file.
>
> Humm, I see. But some files are in a desperate state, needing
> a reindentation and better organization.
>
> The file that I wanted to change had a function with many
> lines (which makes it hard to understand its purpose). Also,
> it had many long lines, which make using patches a little
> harder than if it had shorter lines. It would be better to
> have them neatly organized.
>
> And the biggest problem of all was that the indentation was so
> strange that top-level statments of a function had no
> indentation at all.
>
> I would like to print the code to study it well (perhaps
> documenting it and regarding this as a small-scale "audit" of
> some modules that I understand), but not having indentation
> makes this task take more time. :-(
>
I agree with all your points. Having this 'no cosmetic changes' rule
makes the code go into a unreadable state IMHO. One quite dirty work-
around is to run the code you want to study thru 'indent' or something
similiar and then submit patches made with 'diff -b -B'. This wont
make the code in the cvs repository any more readable though.
/ Fredrik
More information about the MPlayer-users
mailing list