
|
If you were logged in you would be able to see more operations.
|
|
|
|
|
| Component/s: |
sipXtapi
|
| Affects Version/s: |
None
|
| Fix Version/s: |
None
|
|
|
This patch was originally created by Dennis Surovtsev. I did not check is it working or not and how it conform to coding style guideline. Feel free to use it as inspiration or clean up it and resubmit. I would happily check in G.729 and G.723 support if somebody provide good patch.
|
|
Description
|
This patch was originally created by Dennis Surovtsev. I did not check is it working or not and how it conform to coding style guideline. Feel free to use it as inspiration or clean up it and resubmit. I would happily check in G.729 and G.723 support if somebody provide good patch. |
Show » |
|
These are modified patches of Dennis Surovtsev. They contain some bugfixes, code style changes.
G729A works for me using Intel IPP 5.1. I have tested it by dialing another "phone" - ReceiveCall_dbgNoVideo.exe
Then also by dialing a test number of a voip operator, and also by calling another number of the same operator and having a conversation with myself on another phone.
If there is silence, then rtp frames of size 0 or 2 appear, these are rejected, I dont know what to do with these right now.
Also if there is silence, then decoder sometimes decodes 240 bytes instead of 320 bytes (160 samples), but decoder pretends 160 samples were decoded (but there are no audio glitches mysteriously) as the USC decoder doesnt return the number of decoded bytes in a variable. It returns instead number of input bytes processed.
Alexander needs to have a look at this code and maybe fix some bugs or do style changes. It should also be tested by more people than 1.
G723.1 - Dennis wrote the patches as if this decoder decoded both 723.1 and 723 but I have read 723 is completely different so I dont know how that would work. So the code was modified to work with 723.1 only.
There are some changes in the code, but it is untested and probably wont work (729A didnt work in the beginning either). Someone needs to test this. I dont need this codec right now.
The readme.txt contains everything you should need to get it to compile.