> /// Create a VideoDecoder for the specified codec_type > // > /// @return 0 if no decoder could be created for the specified > encoding > /// > virtual std::auto_ptr<VideoDecoder> createVideoDecoder(VideoInfo& > info)=0; > > I like that design in general. > Yes, there might be other exceptions thrown for OOM conditions or similar, > but I'd rather have MediaHandler catch them then MediaHandler users.
Also, I think it's a good design to get something back from a failed createVideoDecoder that tells you what went wrong. After all, it fails because of something passed in by the caller. If you get 0, you have no idea what happened. A descriptive MediaException throw fulfils that. bwy
signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil
_______________________________________________ Gnash-dev mailing list [email protected] http://lists.gnu.org/mailman/listinfo/gnash-dev
