< Previous by Date | Date Index | Next by Date > |
Thread Index |
Thanks Micky, I’ve reviewed all your comments on
the UAS and UAC states. I’ve fixed either the .dot or sources
appropriately (I hope). Changes are in SVN. A lot of the UPDATE in early dialog code
and PRACK code is not completed – so there are some unimplemented states
and transitions. I elected to leave out most of the transitions to Assert
state for simplicity. Let me know if I missed something. Thanks, Scott From:
resiprocate-devel-bounces@xxxxxxxxxxxxxxxxxxx
[mailto:resiprocate-devel-bounces@xxxxxxxxxxxxxxxxxxx] On Behalf Of Micky Kaufmann Attached another file for
‘dum-serverinvite-state.dot’. By the way the comparison between the
state machines and the source was done on svn version 5412 checkout. From:
resiprocate-devel-bounces@xxxxxxxxxxxxxxxxxxx [mailto:resiprocate-devel-bounces@xxxxxxxxxxxxxxxxxxx]
On Behalf Of Hi All, I’ve attached an RTF file containing differences
between the source code and the file ‘dum-clientinvite-state.dot’. You can see in the attached file that in some transitions
the description of what to do is missing a few methods or a method was added to
it. For example: some transitions from the start event are
missing the ‘app::onNewSession’ method, but there’s a
transition from Early to EarlyWithOffer that contains
‘app::onNewSession’ even though it seems an error. In some cases the event causes a transition from one event
to another when the second state seems wrong! Some transitions are missing and I couldn’t find some
of the transitions in the source code. Since I’m not always sure if the fix for an error
should be in the source code or in the state machine (like in the
‘app::onNewSession’ example) I’m posting it to this
newsgroup… For now I only reviewed the client state machine and since
I’m pretty sure the document I’m sending will solve some bugs
I’ll continue to review the other invite state machines. By the way I also attached the draft drawings of the state
machines – all in one visio file. Regards, Micky |