Wireshark-bugs: [Wireshark-bugs] [Bug 5589] DIS update to enable radio voice comms jitter analys

Date: Mon, 17 Jan 2011 05:41:19 -0800 (PST)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5589

--- Comment #6 from Chuck Kristofek <chuck.kristofek@xxxxxxx> 2011-01-17 05:41:17 PST ---
(In reply to comment #5)
> We should look into sharing the stream analysis and player, i.s.o. duplicating
> it.

Jake,

Agreed.  My observations:
1.  I did not want to corrupt the RTP solutions with a fold.  Now that the DIS
solution is working it is easier to see the common code and regression test
both the DIS and RTP products with assurance of not breaking the RTP product.
2.  The DIS voice protocol(w/ no SSRC) is not nearly as robust as RTP and there
is only a one way broadcast paradigm (no reverse).  This makes the analysis of
each much different.
3.  I believe the RTP jitter buffer logic may be flawed.  It relies too much
upon tx timestamps instead of what the receiver is experiencing. Should I post
my concerns to the WS development community?

Finally, I'm in a time pinch for the near future.  The simulator community has
a need for this solution now (last year actually).  Can you accept the patch as
is with the understanding of my future contributions including code
optimizations/reductions?

Kind Regards,
Chuck

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.