SeqManager.cpp: fix num discontinued bug and increase performance

[penguinol]

#395

[ibc]

The test does not compile for me. It looks like you are not synced with v3 branch:

../test/src/RTC/TestSeqManager.cpp:409:15: error: use of class template 'TestSeqManagerInput' requires template
      arguments
                std::vector<TestSeqManagerInput> inputs =

[ibc]

Please take a look to the current TestSeqManager.cpp in mediasoup v3 branch:

• https://github.com/versatica/mediasoup/blob/v3/worker/test/src/RTC/TestSeqManager.cpp

All inputs are created with the proper template syntax:

std::vector<TestSeqManagerInput<uint16_t>> inputs =

However the diff in your PR does not include those templates.

[ibc]

Also, I think SeqMaanger.cpp should include #include <iterator> which exposes std::distance().

[ibc]

Also, I think SeqMaanger.cpp should include #include <iterator> which exposes std::distance().

Please add this and we are done. And let me know your name/URL or something to add into mediasoup Ackknowledges.

[ibc]

Merged, thanks a lot!

[penguinol]

Also, I think SeqMaanger.cpp should include #include <iterator> which exposes std::distance().

Please add this and we are done. And let me know your name/URL or something to add into mediasoup Ackknowledges.

JunLong Zou
www.lonny.me

[ibc]

Thanks, added here: https://mediasoup.org/about/#acknowledgements