[FFmpeg-cvslog] r12241 - trunk/libavformat/mov.c

Reimar Döffinger Reimar.Doeffinger
Wed Feb 27 15:01:03 CET 2008


On Wed, Feb 27, 2008 at 02:35:11PM +0100, Baptiste Coudurier wrote:
> Reimar D?ffinger wrote:
> > On Wed, Feb 27, 2008 at 01:35:29PM +0100, Baptiste Coudurier wrote:
> >> The problem is that you don't see any use of this feature, and fail to
> >> understand why it is useful, but you should first read specs of 'alis'
> >> and Alias Record functionnality.
> > 
> > Hm. I only read parts of the Quicktime format specification and could
> > find very little about it, but I noticed that it says:
> > "A value of 'alis' indicates a file system alias record." (Chapter 2,
> > page 91, "Data Reference Atom",
> > http://developer.apple.com/documentation/QuickTime/QTFF/qtff.pdf).
> > In which case your implementation is just plain broken, since a value of
> > "http://test.com/test" then would have to open the _file_ named thus,
> > whereas your implementation would try to open a http connection.
> > And the generated filename indeed can start with http:// since the
> > "volume" part at least in the current implementation can contain any
> > string.
> 
> Please read code and specs another time, and search deeper.

Look, if you want to tell me that I am an idiot and the code the code replaces
all ':' by '/' and thus url_fopen will only open local files, then just
say so (and preferably add a comment to the corresponding url_fopen).
Most of the problems are then not actually there, unless and until
someone comes along and thinks: "hey, why not just implement the url
reference type as well, it's almost a one-liner".
Though since I have not found anything so far specifying what an
"Macintosh alias" is I wonder how if at all filenames with a ':' are
supposed to be specified...




More information about the ffmpeg-cvslog mailing list