[MPlayer-dev-eng] Indentation style (was: [PATCH] Fix useless dependencies in the stream lib)

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Thu Apr 10 11:14:56 CEST 2008


On Thu, Apr 10, 2008 at 11:03:53AM +0200, Alban Bedel wrote:
> On Thu, 10 Apr 2008 02:15:30 +0200
> Diego Biurrun <diego at biurrun.de> wrote:
> > Not at all.  Tags would clutter up each individual source file.
> > Resource files would not even need to be part of the MPlayer
> > repository.
> 
> But your wording seems to imply you don't want them in the repo, or?
> Sure I can do whatever I want in my $HOME, but if they can't be
> published with the rest of the MPlayer code, that severely reduce their
> usefulness.

I don't think it would hurt to have them e.g. in DOCS/. I wouldn't mind
them too much in source files either, though they clutter things up a
bit.
There is a bit of a problem of maintaining though, if it is included in
SVN it should be commented so that at least the most basic changes are
possible without further knowledge.
And I am still in favour of the ffmpeg commit hooks for the files I
maintain, while there have been objections about such scripts, I don't
think any of the reasons apply to the files I maintain at least as long
as I am actively maintaining them and available on short notice.

Greetings,
Reimar Döffinger



More information about the MPlayer-dev-eng mailing list