Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
tsfix: fix pts/dts for all streams (including teletext)
  • Loading branch information
perexg committed Oct 23, 2014
1 parent 4ad249e commit 6106b71
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/plumbing/tsfix.c
Expand Up @@ -154,7 +154,7 @@ normalize_ts(tsfix_t *tf, tfstream_t *tfs, th_pkt_t *pkt)

int checkts = SCT_ISAUDIO(tfs->tfs_type) || SCT_ISVIDEO(tfs->tfs_type);

if (checkts || SCT_ISSUBTITLE(tfs->tfs_type)) {
if (1 /* all streams */) {
if(tf->tf_tsref == PTS_UNSET) {

This comment has been minimized.

Copy link
@ksooo

ksooo Oct 23, 2014

Contributor

@perexg: if (1) ... ugly. Do you plan to remove this?

This comment has been minimized.

Copy link
@perexg

perexg Oct 23, 2014

Author Contributor

Removed in c297aa7 .

pkt_ref_dec(pkt);
return;
Expand Down

3 comments on commit 6106b71

@johsmi
Copy link
Contributor

@johsmi johsmi commented on 6106b71 Oct 23, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@perexg : I presume this reintroduces bug #2178 that I tried to address in commit 92b1a08? Is this supposed to be temporary only or do we need to find another solution to #2178?

@perexg
Copy link
Contributor Author

@perexg perexg commented on 6106b71 Oct 23, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@johsmi : It must be reworked. I tried to create a simple hack in 9c6b8c2 . Could you check, review, test ?

@johsmi
Copy link
Contributor

@johsmi johsmi commented on 6106b71 Oct 23, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@perexg : Thank you. 9c6b8c2 seems to work and the logic looks good to me. I'll add a question in the commit as a line note.

Please sign in to comment.