[MPlayer-dev-eng] Re: [PATCH] make demuxer_get_percent_pos() recheck file length
Oded Shimon
ods15 at ods15.dyndns.org
Mon Oct 3 19:55:16 CEST 2005
On Mon, Oct 03, 2005 at 12:55:33PM -0500, Joey Parrish wrote:
> On Mon, Oct 03, 2005 at 09:07:31AM +0300, Oded Shimon wrote:
> > On Sun, Oct 02, 2005 at 10:56:05PM -0700, Scott W. Larson wrote:
> > > Alban Bedel wrote:
> > > >The fd field could be anything (ie there not even a guarentee that
> > > >it acctually hold a fd),
> > >
> > > I was trusting this comment in libmpdemux/stream.h.:
> > >
> > > typedef struct stream_st {
> > > :
> > > int fd; // file descriptor, see man open(2)
> > > :
> >
> > A stream can be anything.. what about windows, if i'm not mistaken windows
> > doesn't even have such a thing... playing with this fd is like playing with
> > the internals of something you shouldn't be touching..
>
> If you're suggesting that windows doesn't have file descriptors, then
> you're wrong.
I dunno anything about windows API, but does it have integer file
descriptors?.. as far as i know it doesn't... either way, these veriable
should not be used from this function..
- ods15
More information about the MPlayer-dev-eng
mailing list