I'm not a core dev, but my guess would be to reduce the risk of breaking current installations while mod_json_cdr was being developed. I do see you point but it does make more sense to move it out. perhaps the common code could be librarified or they could be merged later on, but not breaking current installations is a pretty high priority. This is what happened last time a Linux kernel dev broke something big: <a href="http://goo.gl/u8GWl">http://goo.gl/u8GWl</a><div>
<br><div class="gmail_quote">On 2 January 2013 12:19, Cal Leeming [Simplicity Media Ltd] <span dir="ltr"><<a href="mailto:cal.leeming@simplicitymedialtd.co.uk" target="_blank">cal.leeming@simplicitymedialtd.co.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<div><br></div><div>Looking at the source code for xml_cdr and json_cdr:</div><div><br></div><div><a href="https://github.com/FreeSWITCH/FreeSWITCH/blob/master/src/mod/event_handlers/mod_json_cdr/mod_json_cdr.c" target="_blank">https://github.com/FreeSWITCH/FreeSWITCH/blob/master/src/mod/event_handlers/mod_json_cdr/mod_json_cdr.c</a></div>
<div><a href="https://github.com/FreeSWITCH/FreeSWITCH/blob/master/src/mod/xml_int/mod_xml_cdr/mod_xml_cdr.c" target="_blank">https://github.com/FreeSWITCH/FreeSWITCH/blob/master/src/mod/xml_int/mod_xml_cdr/mod_xml_cdr.c</a></div>
<div><br>
</div><div>Based on the commit messages and file contents, it looks like this code is maintained in two different places, and is mostly copy/paste.</div><div><br></div><div>Looking at the documentation, all the relevant channel information is inside the xml_cdr docs, with very little inside json_cdr, despite them having nearly the same code base.</div>
<div><br></div><div>Could a core dev comment on;</div><div><br></div><div>* Why this code is maintained in two different places - are there any plans to merge it into a single lib/file?</div><div>* Is there functionally any difference between the two, other than the fact one outputs XML and another outputs JSON (my C is very poor, so I find it difficult to interpret the code logic - sorry)</div>
<div><br></div><div>Many thanks</div><span class="HOEnZb"><font color="#888888"><div><br>Cal</div>
</font></span><br>_________________________________________________________________________<br>
Professional FreeSWITCH Consulting Services:<br>
<a href="mailto:consulting@freeswitch.org">consulting@freeswitch.org</a><br>
<a href="http://www.freeswitchsolutions.com" target="_blank">http://www.freeswitchsolutions.com</a><br>
<br>
FreeSWITCH-powered IP PBX: The CudaTel Communication Server<br>
<a href="http://www.cudatel.com" target="_blank">http://www.cudatel.com</a><br>
<br>
Official FreeSWITCH Sites<br>
<a href="http://www.freeswitch.org" target="_blank">http://www.freeswitch.org</a><br>
<a href="http://wiki.freeswitch.org" target="_blank">http://wiki.freeswitch.org</a><br>
<a href="http://www.cluecon.com" target="_blank">http://www.cluecon.com</a><br>
<br>
FreeSWITCH-users mailing list<br>
<a href="mailto:FreeSWITCH-users@lists.freeswitch.org">FreeSWITCH-users@lists.freeswitch.org</a><br>
<a href="http://lists.freeswitch.org/mailman/listinfo/freeswitch-users" target="_blank">http://lists.freeswitch.org/mailman/listinfo/freeswitch-users</a><br>
UNSUBSCRIBE:<a href="http://lists.freeswitch.org/mailman/options/freeswitch-users" target="_blank">http://lists.freeswitch.org/mailman/options/freeswitch-users</a><br>
<a href="http://www.freeswitch.org" target="_blank">http://www.freeswitch.org</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><b>Andrew Cassidy BSc (Hons) MBCS SSCA</b><div>Managing Director<div><div><img src="http://c1170247.r47.cf3.rackcdn.com/emailsig.png"><br></div><div><br><div>
<b style="color:rgb(51,135,171);text-decoration:none;font-family:sans-serif"><a href="mailto:info@cassidywebservices.co.uk" style="color:rgb(51,135,171);text-decoration:none;font-family:sans-serif" target="_blank">T</a> </b>03300 100 960
<b style="color:rgb(51,135,171);text-decoration:none;font-family:sans-serif"><a href="mailto:info@cassidywebservices.co.uk" style="color:rgb(51,135,171);text-decoration:none;font-family:sans-serif" target="_blank">F</a> </b>03300 100 961</div>
<div><b style="text-decoration:none;font-family:sans-serif"><a href="mailto:info@cassidywebservices.co.uk" style="color:rgb(51,135,171);text-decoration:none;font-family:sans-serif" target="_blank">E</a> </b><a href="mailto:andrew@cassidywebservices.co.uk" target="_blank">andrew@cassidywebservices.co.uk</a></div>
<div><b style="text-decoration:none;font-family:sans-serif"><a href="mailto:info@cassidywebservices.co.uk" style="color:rgb(51,135,171);text-decoration:none;font-family:sans-serif" target="_blank">W</a> </b><a href="http://www.cassidywebservices.co.uk" target="_blank">www.cassidywebservices.co.uk</a></div>
</div></div></div>
</div>