Hi Kieran, On Thu, Aug 10, 2023 at 9:09 AM Kieran Kunhya <kier...@obe.tv> wrote: > You're exposing this incorrect information as public API, two wrongs don't > make a right.
The information is already exposed via the public API, in the form of pkt->pts. The fact that I've exposed that same value as side data as well doesn't really change that situation. > I also told the author of the previous code that it was wrong but the patch > was forced through on the guise that "professionals won't respect ffmpeg if > scte-35 isn't demuxed". So you're annoyed that seven years ago somebody else submitted some patch and it got merged over your objection? And that in that time nobody (including you) has cared enough to take it upon themselves to improve the accuracy of the timestamp? > The fact something isn't used often, doesn't mean it should be implemented > badly. You could say that interlaced isn't used much as a total of all the > video in the world so we should just not decode it correctly. I've made no attempt to argue that the accuracy of the timestamp couldn't be improved. I would argue that improving it shouldn't be any sort of prerequisite to merging this patch series though. Presumably it hasn't happened already because 1) properly implementing PCR interpolation within the ffmpeg mpegts demux implementation is difficult, 2) nobody has cared enough to do it, and/or 3) the typical PCR interval is 40ms so people are happy with it being "close enough" in the splice immediate case. > By all means keep your hacks in your forks. Where is the hack? My patches are an incremental improvement over what is already there. The notion that it makes no attempt to solve some orthogonal problem that I'm not really worried about isn't grounds for rejecting it. I would be happy to see other developers weigh in, or if Kieran's is the final word then I would like to see this reviewed by the TC. I'm making a genuine effort to improve the support in ffmpeg and it seems like an artificial barrier is being raised against those efforts. Devin Devin -- Devin Heitmueller, Senior Software Engineer LTN Global Communications o: +1 (301) 363-1001 w: https://ltnglobal.com e: devin.heitmuel...@ltnglobal.com _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
