Bug 151 - recon reflow: generalize code for access to media stack
Summary: recon reflow: generalize code for access to media stack
Status: RESOLVED FIXED
Alias: None
Product: reSIProcate
Classification: Unclassified
Component: recon (librecon) (show other bugs)
Version: unspecified
Hardware: All All
: P1 enhancement
Assignee: Owner of all unassigned bugs
URL:
Keywords:
Depends on:
Blocks: 93 145 146
  Show dependency treegraph
 
Reported: 2021-04-08 05:47 CDT by Daniel Pocock
Modified: 2021-07-01 05:20 CDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Pocock 2021-04-08 05:47:09 CDT
Code relating to sipXtapi is mainly in reflow and there are some traces of it in resip/recon

This code needs to be generalized to handle any arbitrary media stack before exploring some of the specific media stacks suggested in their own bugs.

Some initial work on this began here on a branch:

https://github.com/resiprocate/resiprocate/tree/b-counterpath-recon-20130424

After looking at the code, I made some general observations about how to proceed:

- check if any other branches in the repository have anything for reflow or recon

- begin with generalizing the code in resip/recon/*, there are less references to sipX here

- make a table or diagram showing which classes touch sipX and the corresponding API on each side, macro names, variable names, class names and method names

- add an extra column to the table or diagram suggesting how each of these can be generalized

- add an extra column to the table or diagram with equivalent API method names from one of the other media stacks
Comment 1 Daniel Pocock 2021-07-01 05:20:19 CDT
Mostly done now

Since this commit, we can build resip/recon without having the sipXtapi headers present.

https://github.com/resiprocate/resiprocate/commit/2a43a7f2a5cbf9ea05fad9a1d592f19375b24a20