[MPlayer-dev-eng] Offer of help to get me started

Evil Kosh evil_kosh_uk at yahoo.co.uk
Mon Mar 24 01:59:50 CET 2003


On Mon, 2003-03-24 at 00:55, Anders Rune Jensen wrote:
> Evil Kosh <evil_kosh_uk at yahoo.co.uk> writes:
> 
> > Hi,
> >
> > 	I'd like to start a bit of coding on mplayer and I thought the best way
> > to get acquainted with the code and how it works is to look through it
> > and see what I could fix.  I immediately saw that there were a number of
> > different styles that people were developing with.
> >
> > 	Some people would use braces with for and if statements, others would
> > not, some people would put the break in a case statement before the
> > closing brace, others would not.  I thought perhaps it would be a good
> > idea to get acquainted with the code and help out at the same time. 
> > Also I found some parts of the code with tabbed indentations and others
> > with space indentations.
> >
> > 	Would anyone here object to a clear style guide being formed from the
> > developers here?  I'd be happy to reformat the code along those
> > guidelines, if it means I can help out, everyone gets clear, well
> > formatted code along an agreed style.
> >
> > What do you think?  well, I'm 1/2 way through reformatting mplayer.c
> > with what I think is a good style.  It's consistant anyhow.  So if
> > people dont like it then I can always alter it.  I'll post it to the
> > list when I've finished and let people judge for themselves whether they
> > think this reformatting exercise is a good idea or not.
> 
> Ever heard of pretty printers?

I think I understand what you are trying to say.  There is however a
limit to the amount of automated reformatting tools can do before you
have to manually go in and change the formatting in such a way you can
rerun the automated reformatting tool again, or complete it manually.

I'm simply suggesting that people agree on one style and it's something
I can help implement.  I'm not trying to make the source beautiful, I'm
trying to smooth the code out so it's more readable and accessible.  It
was my first thought when I read the code, that it was unreadable in
certain parts.  So the first thing I would like to help with, is making
it more accessible to people like me.

> > thanks for listening and I hope a favourable response.
-- 
Evil Kosh <evil_kosh_uk at yahoo.co.uk>



More information about the MPlayer-dev-eng mailing list