The current implementation of IDirectFB::CreateVideoProvider checks that the file referenced by the supplied filename is readable. The Xine video provider is capable of handling Xine 'MRLs' as is but this test in CreateVideoProvider prevents anything that isn't a plain filename getting through to the Xine provider's Probe method.
Would it be reasonable to remove this test in CreateVideoProvider? This would mean that 'file not found' and similar errors are caught later on and that the providers will be probed in this case, but this only creates a performance hit for something that's already an error condition. The advantage is that all the input options supported by Xine become instantly available to DirectFB apps.
I don't think there are any side effects to this change: all file access errors will still be caught, either by the fact that no provider's Probe method accepts the file or because the Construct method later fails.
Mark
_______________________________________________ directfb-users mailing list [email protected] http://mail.directfb.org/cgi-bin/mailman/listinfo/directfb-users
