Complete re-implementation of sdp_remove_line_by_prefix(), maybe reviewing by diff does not make sense. Also feedback is on unit test is welcome as I have not created such for Kamailio earlier. You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/522
-- Commit Summary --
* modules/sdpops: reimplement sdp_remove_line_by_prefix() to fix shortcomings * test/unit: add unit test for sdpops script function sdp_remove_line_by_prefix()
-- File Changes --
M modules/sdpops/sdpops_mod.c (118) A test/unit/60-message-sdp0.sip (18) A test/unit/60-message-sdp1.sip (18) A test/unit/60-message-sdp2.sip (19) A test/unit/60-message-sdp3.sip (19) A test/unit/60-message-sdp4.sip (20) A test/unit/60-message-sdp5.sip (20) A test/unit/60-message-sdp6.sip (20) A test/unit/60-message-sdp7.sip (19) A test/unit/60.cfg (41) A test/unit/60.sh (75) A test/unit/include/require.sh (65)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/522.patch https://github.com/kamailio/kamailio/pull/522.diff
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/522