<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN"><html><head><meta http-equiv="Content-Type" content="text/html;charset=utf-8"></head><body style="font-family: HelveticaNeue-Light, Helvetica, Arial;font-size: 13pt !important;">
                    <style type="text/css">
                        #INCREDITEXTREGION {
                            white-space: normal !important;
                        }
                        #INCREDITEXTREGION * {
                                white-space: inherit !important;
                        }
                        #INCREDITEXTREGION a {
                            word-break: break-all !important;
                        }
                        @media (max-width: 350px) {
                                div[data-imsnippettype=narrow] {
                                        display: block !important;
                                        max-height:none !important;
                                        overflow:visible !important;
                                        margin-top:10px !important;
                                        margin-bottom:38px !important;
                                }
                                div[data-imsnippettype=wide] {
                                        display: none !important;
                                }
                        }
                    </style>
                <div id="newConversation">
                    <div id="templateData">
                        <div id="INCREDITEXTREGION" dir="auto">
                        </div>
                    </div>
                    <div>
                        <br>
                    </div>
                    <div class="im_signature" id="im_signature" style="font-size: 12pt; margin-bottom: 12px; display: none; " dir="auto"></div>
                    <div class="im_stamp" id="im_stamp" dir="auto" style="font-size:12pt">r_frame_rate si a deprecated field and its interpretation is 99% wrong. <span style="font-size: 12pt; ">You should use avg_frame_rate instead.</span><div><span style="font-size: 12pt; "><br></span></div><div><span style="font-size: 12pt; ">Regarding the question, time base is usually a more accurate source of information.</span></div><div><span style="font-size: 12pt; "><br></span></div><div><span style="font-size: 12pt; ">What problem were you trying to resolve?</span></div><div><span style="font-size: 12pt; ">Vittorio</span></div></div>
                </div>
                <div>
                    <br>
                </div>
            <div></div><div id="im_ipad_separator"></div><div class="metadata"><div><span style="color: #606060;font-size: 12pt;">From:</span><span style="font-size: 12pt"> 某因幡</span></div><div><span style="color: #606060;font-size: 12pt;">Sent:</span><span style="font-size: 12pt"> lunedì, marzo 10, 2014 09:22 AM</span></div><div><span style="color: #606060;font-size: 12pt;">To:</span><span style="font-size: 12pt"> Mailing list for VLC media player developers</span></div><div><span style="color: #606060;font-size: 12pt;">Subject:</span><span style="font-size: 12pt"> [vlc-devel] Question about video FPS probing in AVFormat demuxer</span></div><br></div><div class="previous-conversation"><div dir="ltr"><div>Hi,<br><br></div>As the code piece below shows, when creating new ES, why using time base from codec not in stream, or it means something else here?<br><br><br>--- a/modules/demux/avformat/demux.c<br>+++ b/modules/demux/avformat/demux.c<br>
@@ -367,8 +367,8 @@ int OpenDemux( vlc_object_t *p_this )<br> # warning FIXME: implement palette transmission<br> #endif<br>             psz_type = "video";<br>-            fmt.video.i_frame_rate = cc->time_base.den;<br>
-            fmt.video.i_frame_rate_base = cc->time_base.num * __MAX( cc->ticks_per_frame, 1 );<br>+            fmt.video.i_frame_rate = s->r_frame_rate.den;<br>+            fmt.video.i_frame_rate_base = s->r_frame_rate.num;<br>
             fmt.video.i_sar_num = s->sample_aspect_ratio.num;<br>             if (s->sample_aspect_ratio.num > 0)<br>                 fmt.video.i_sar_den = s->sample_aspect_ratio.den;<br clear="all"><div><div>
 <br>-- <br>language: Chinese, Japanese, English
</div></div></div>

</div></body></html><meta name="viewport" content="width=device-width">