[MPlayer-dev-eng] squelch some warnings (patches incl.)
Michael Niedermayer
michaelni at gmx.at
Fri Feb 29 00:48:36 CET 2008
On Tue, Feb 26, 2008 at 07:00:03PM +0100, Diego Biurrun wrote:
> On Mon, Feb 25, 2008 at 03:20:20AM +0100, Michael Niedermayer wrote:
> >
> > Cleanup to snow is welcome, some examples:
> > * normalize {} placement
>
> Elaborate what you mean, this is ambiguous, i.e. I'm not sure what your
> preference is.
NOT:
for(x=0; x<w; x++)
{
line[x] -= 128 << FRAC_BITS;
// buf[x + y*w]-= 128<<FRAC_BITS;
}
but:
for(x=0; x<w; x++){
line[x] -= 128 << FRAC_BITS;
// buf[x + y*w]-= 128<<FRAC_BITS;
}
but please dont change {} for functions, they can be:
static void slice_buffer_flush(slice_buffer * buf)
{
int i;
for (i = 0; i < buf->line_count; i++){
if (buf->line[i])
slice_buffer_release(buf, i);
}
}
>
> The rest is done,
not really:
return;
}
for(mb_x=0; mb_x<=mb_w; mb_x++){
add_yblock(s, 1, sb, old_buffer, dst8, obmc,
----
if(sliced){
s->dsp.inner_add_yblock(obmc, obmc_stride, block, b_w, b_h, src_x,src_y, src_stride, sb, add, dst8);
}else
for(y=0; y<b_h; y++){
...
> your turn now :)
done
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Thouse who are best at talking, realize last or never when they are wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20080229/f7c0e266/attachment.pgp>
More information about the MPlayer-dev-eng
mailing list