Kenny Shumard wrote:

I've finished a working version of the IAX specification and posted it

First--thanks for the effort, I know a lot of people have been waiting for this.


From a quick read, one thing jumped out at me. Section 5 ("Features") seems very out of place in the flow of the document. The implementation descriptions dive into the use of IAX frame types which aren't introduced until section 8. So, I'd recommend either moving the whole thing there (after 8), or perhaps leaving it in place, but up-leveling it to describe the feature capability without going into the protocol details at that point.

A state machine or transaction waterfall diagram would then be useful for each of the feature subsections in 5, to describe the detailed protocol. Perhaps adding these diagrams to the existing section 12 ("Call types") would be appropriate.

Finally, minor nit--I didn't see reference to this as the 2nd version of IAX in use, was that intentional?

Overall--great first pass.

-Johnathan
_______________________________________________
Asterisk-Dev mailing list
Asterisk-Dev@lists.digium.com
http://lists.digium.com/mailman/listinfo/asterisk-dev
To UNSUBSCRIBE or update options visit:
  http://lists.digium.com/mailman/listinfo/asterisk-dev

Reply via email to