[MPlayer-dev-eng] [PATCH] Correct format conversion in stream_vstream.c
Reimar Döffinger
Reimar.Doeffinger at stud.uni-karlsruhe.de
Fri Aug 24 07:46:53 CEST 2007
Hello,
On Fri, Aug 24, 2007 at 01:20:46AM +0200, Carl Eugen Hoyos wrote:
> Attached patch should fix one warning concerning format conversion in
> stream_vstream.c. Is %"PRIu64" correct for uint64_t (stream->end_pos)?
Yes, but
1) stream->end_pos is off_t, which is not necessarily uint64_t
2) The original was %d, so it should be PRId64 at least for now.
About the off_t problem, either cast stream->end_pos to int/unsigned or
do the PRId64 change and cast it to uint64_t.
A patch that replaces off_t by something like libav* offset_t that is
always 64 bit IMO would be welcome, too.
Greetings,
Reimar Döffinger
More information about the MPlayer-dev-eng
mailing list