[MPlayer-dev-eng] Re: [PATCH] libmpdemux/asf_streaming.c

Björn Sandell biorn at dce.chalmers.se
Fri Oct 24 16:24:50 CEST 2003


On Fri, 24 Oct 2003 13:43:21 +0200
Tobias Diedrich <ranma at gmx.at> wrote:

> Björn Sandell wrote:
> 
> > I think this is nicer and more consistent with similar code elsewhere in the
> > three. The '-1' in 'sizeof(buf)-1' is not needed (per SUS) but as it's used
> > at other places I used it here as well.
> 
> Wouldn't it be better to fix all the occurences, where sizeof(buf)-1 is
> used even though sizeof(buf) is perfectly safe?

It would, if it really is perfectly safe on _all_ supported platforms.

-- 
Björn Sandell       DCE/DFS Sysadmin       IT department
Chalmers University of Technology    www.dce.chalmers.se



More information about the MPlayer-dev-eng mailing list