[MPlayer-users] Patches for changing indentation accepted?
Rogério Brito
linuxconsult at yahoo.com.br
Sun May 5 11:58:02 CEST 2002
Well, it seems that some parts of mplayer still have problems
with endianness.
While trying to fix mplayer to work with Linux under PowerPC,
I tried to read the source code and I saw that some files are
not well indented (for example, libmpdemux/aviheader.c),
making it hard to understand and to maintain.
It is said in DOCS/tech/patches.txt that patches that change
indentation would not be acceptable, BUT are there exceptions
for important cases (like for the file above, which, I'm sure
other people will agree, is quite hard to maintain)?
Also, I have other questions about the development process:
1 - how is the development model of mplayer? Since the project
is in a pre-release phase, is it currently in a feature
freeze? When will further changes be accepted?
2 - are there any coding standards to follow? Are there plans
have one, to increase the maintainability and modularity
of the project as it grows?
3 - regarding quality assurance (and also connected with
question 1), is there any standard test suite used to make
sure mplayer works correctly with (at least) a know set of
files?
I'm asking all this because I already submited one patch for
corrections of endianness problems in a PowerPC processor and
I would like to contribute further if it is wanted.
Thanks in advance, Roger...
--
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
Rogério Brito - rbrito at iname.com - http://www.ime.usp.br/~rbrito/
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
More information about the MPlayer-users
mailing list