[MPlayer-dev-eng] [PATCH?] [RFC] Script to check "sanity" of MPlayer sources

[Ivo]

On Saturday 06 August 2005 10:26, Torinthiel wrote:
> On Sat, Aug 06, 2005 at 02:07:56AM +0200, Ivo wrote:
> >         -(no)extensions    test for uppercase extensions [default: yes]
>
> AFAIK .s is something different from .S

Can you tell me what the difference is?

If I change loader/wrapper.S to loader/wrapper.s, make calls:

as   -o wrapper.o wrapper.s

instead of:

cc    -c -o wrapper.o wrapper.S

The resulting .o file (of as) is exactly 100 bytes smaller. There doesn't 
seem to be any functional difference AFAICT.

> >         -(no)rcsid         test for missing RCS Id's [default: no]
>
> And you probably don't take into account the $Revision$ tags in some
> files, especially documentation?

No, but I could change that and grep for $Id...$ or $Revision...$.

> But overall the script seems a nice idea for me, probably even
> extendible. I once wrote a script for checking whether the documentation
> is in sync, could be for example added to this.

Yes. If people think it's a good idea to add this to cvs, feel free to 
extend it ofcourse :) Running xmllint on all .xml files might also be a 
good test.

I just found out that the -trailws and -rcsid numbers given in an earlier 
post were false. The script also processes all .#* floating around. That 
should be fixed too :)

--Ivo