[MPlayer-cvslog] r34746 - trunk/libmpdemux/demux_lavf.c

[reimar]

Author: reimar
Date: Sun Feb 19 21:33:32 2012
New Revision: 34746

Log:
demux_lavf: Explain under which conditions to add demuxers to preferred_list.

Modified:
   trunk/libmpdemux/demux_lavf.c

Modified: trunk/libmpdemux/demux_lavf.c
==============================================================================
--- trunk/libmpdemux/demux_lavf.c	Sun Feb 19 21:01:17 2012	(r34745)
+++ trunk/libmpdemux/demux_lavf.c	Sun Feb 19 21:33:32 2012	(r34746)
@@ -207,6 +207,14 @@ static int lavf_check_file(demuxer_t *de
     return DEMUXER_TYPE_LAVF;
 }
 
+/* Before adding anything to this list please stop and consider why.
+ * There are two good reasons
+ * 1) to reduce startup time when streaming these file types
+ * 2) workarounds around bugs in our native demuxers that are not reasonable to
+ *    fix
+ * For the case 2) that means the issue should be understood well
+ * enough to be able to decide that a fix is not reasonable.
+ */
 static const char * const preferred_list[] = {
     "cdxl",
     "dxa",