PLI dilution (Multistream)

[natikaltura]

Implementation of PR #3422 (#3422) for multi-stream

Would love to get your input.
Nati Baratz

[natikaltura]

Hi @lminiero,

I had some git issues so sorry for the mess.
Fixed the 2 PR’s.

Reopening the code for this PR.

Thanks

[lminiero]

Your PR still includes the changes to the management of simulcast, which has nothing to do with the main purpose of the patch. I already explained that you can already configure the behaviour per subscriber using the API, and that publisher based APIs for framerate can be done in future efforts.

[natikaltura]

@lminiero I'm a bit confused :)
The changes doesn’t alter any functionality related to simulcast management.
I believe what you're referring to is the log in rtp.c, which simply adds an additional field (Milliseconds) to the log. I find this addition very helpful for tracing PLI handling.
Cheers

[lminiero]

Added a few editorial notes. I guess most of them will apply to the 0.x version as well, since they're "twins". Thanks!

[natikaltura]

"Hi @lminiero, thanks for your comments! All fixed and pushed.

[lminiero]

Thanks for the fixes! I still see a couple of minor things to adjust, please see inline.

[natikaltura]

@lminiero, done, thanks! I believe next time will go smoother! :)

[natikaltura]

@lminiero, fixed..

[lminiero]

Thanks for your work and your patience, merging!