PrePostRollFlagger results sometimes wrong

[T.M.Speight.90@…]

I don't think I need to describe the symptoms with this patch, as it's clearly a copy/paste error!

In short, I sometimes end up with skiplists such as:

mysql> select mark,SEC_TO_TIME(mark/25) AS time,type,data from recordedmarkup where chanid=1002 and starttime ='2011-10-25 00:29:00' AND type IN (2,4,5) order by mark;

+-------+----------+------+------+
| mark  | time     | type | data |
+-------+----------+------+------+
|     0 | 00:00:00 |    4 | NULL |
|  1581 | 00:01:03 |    4 | NULL |
| 98910 | 01:05:56 |    5 | NULL |
+-------+----------+------+------+

(The start of the second break has been set to the end of the first one, overwriting it). After applying this patch and re-running commflag, I get more sensible results:

+-------+----------+------+------+
| mark  | time     | type | data |
+-------+----------+------+------+
|     0 | 00:00:00 |    4 | NULL |
|  1581 | 00:01:03 |    5 | NULL |
| 91682 | 01:01:07 |    4 | NULL |
| 98910 | 01:05:56 |    5 | NULL |
+-------+----------+------+------+

[Github]

Correct copy/paste error in postroll commercial detection

Corrects an incorrect variable use that was missed when copying a similar block of code added in 9f5ee51db82.

Fixes #10120 Refs #6889

Signed-off-by: Raymond Wagner <rwagner@…>

Branch: master Changeset: 39c61832fb8e9e17fecf478426e67a223f6bd008

[Raymond Wagner]

Backported to 0.24

Branch: fixes/0.24 Changeset: fdfc989ebfe161fd58d977b38a4f2dd75aff1880

[T.M.Speight.90@…]

Thank you for the very fast apply, and for backporting into 0.24. This is what makes Free Software the best! :-)

[stuartm]

Milestone 0.24.2 deleted