rohc team mailing list archive

[Didier Barvaux <question675656@xxxxxxxxxxxxxxxxxxxxx>]

Question #675656 on rohc changed:
https://answers.launchpad.net/rohc/+question/675656

Didier Barvaux posted a new comment:
Hello,

Many thanks for your questions. You discover problems in legacy code,
that's great! :-)

You are right about the RTP SSRC field. The field cannot change for one
given ROHC stream, so the check for changes is useless. I removed it on
master branch. See https://github.com/didier-
barvaux/rohc/commit/ddd5abb6be35ec6bf2c83cd7684b37fa8093c1c0

The logic behind the DF, RND, NBO and SID fields is not defined well.
Currently packet type IR-DYN is forced if there are 3 or more "dynamic
fields" that changed. However, UO packets with Extension 3 can transmit
changes for TTL, TOS, DF, NBO, RND. Those changes are counted as 4
changed fields, so UO with EXT3 is not used while it could be. It might
save some bits. The logic behind dynamic fields should be reworked. I
tried to do that on the master branch. See https://github.com/didier-
barvaux/rohc/commit/ce2d85396ee0b18e7ce376d28aae12f1b83806a0

Please give the updated master branch a try, and tell me how it behaves
for you.

Regards,
Didier

-- 
You received this question notification because your team ROHC Team is
an answer contact for rohc.