[FFmpeg-devel] [PATCH 11/17] lavc/dcaenc: fix make checkheaders.
michaelni at gmx.at
Thu May 10 12:24:42 CEST 2012
On Wed, May 09, 2012 at 11:16:35PM +0200, Clément Bœsch wrote:
> On Wed, May 09, 2012 at 03:33:10PM +0200, Michael Niedermayer wrote:
> > On Wed, May 09, 2012 at 01:25:39PM +0200, Clément Bœsch wrote:
> > > On Wed, May 09, 2012 at 12:45:07PM +0200, Michael Niedermayer wrote:
> > > > On Wed, May 09, 2012 at 10:01:36AM +0200, Clément Bœsch wrote:
> > > > > ---
> > > > > libavcodec/dcaenc.h | 2 ++
> > > > > 1 file changed, 2 insertions(+)
> > > >
> > > > does it make sense to run check headers on non public headers ?
> > > >
> > >
> > > It's certainly simpler to check them all. Do you think all these checks
> > > are a problem?
> > I think the problem lies with how the checks are seen.
> > What we need to do is fix bugs, improve maintainability and readability
> > the checks can point to problems, these problems should be fixed.
> > but when the check points to correct code care must be taken not to
> > worsen the code for no other point than to hide the warning.
> > Here the check should be fixed if no trivial change to the code can
> > solve it.
> > For example i dont think moving an include from the top of the file
> > to the middle is really a good idea for maintainability.
> I agree, and sent a new version of the lavu/error.h patch.
> > Also i think os2threads should fail on non os2 systems, it could be
> > added to SKIPHEADERS if the OS is not OS2. adding ifdefs into it
> > so it can be included but does nothing at all on other platforms
> > doesnt seem like a good idea to me
> Agree too, and new version sent for this one too.
> About the dcaenc.h patch, is there something wrong with it?
if my reply sounded like a objection please disregard it.
about the patchset in general, what i forgot is that i think
checkheaders should be treated like warnings not errors if it is added
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
The real ebay dictionary, page 1
"Used only once" - "Some unspecified defect prevented a second use"
"In good condition" - "Can be repaird by experienced expert"
"As is" - "You wouldnt want it even if you were payed for it, if you knew ..."
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel