[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