Attention is currently required from: dexter.
Patch set 4:Code-Review +1
4 comments:
Patchset:
I expect you did run BTS_Tests/MGCP_Tests/etc. here to validate everything is fine :)
File library/RTP_Emulation.ttcn:
Patch Set #4, Line 364: private function f_check_fixed_rx_payloads(octetstring rtp_data, INT7b rtp_payload_type) runs on RTP_Emulation_CT {
it feels a bit strange having the rtp_Data before the rtp_payload_type, but not criticial.
Patch Set #4, Line 376: if (rtp_data == g_cfg.rx_payloads[i].fixed_payload and rtp_payload_type == g_cfg.rx_payloads[i].payload_type) {
split into 2 lines?
Patch Set #4, Line 388: if (payload_present and payload_error) {
This function implementation seem overcmplex with all those bool vars. Why aren't you incrementing the counters in the respective paths in the loop above? it would make everything more simpler afaict.
To view, visit change 32153. To unsubscribe, or for help writing mail filters, visit settings.