lmao - that is a pretty funny thread.<div><br></div><div>Just realised JIRA is the best place for this - ticket raised;</div><div><a href="http://jira.freeswitch.org/browse/FS-4983">http://jira.freeswitch.org/browse/FS-4983</a></div>
<div><br></div><div>Thanks</div><div><br></div><div>Cal</div><div><br></div><div><div class="gmail_quote">On Wed, Jan 2, 2013 at 12:47 PM, Andrew Cassidy <span dir="ltr">&lt;<a href="mailto:andrew@cassidywebservices.co.uk" target="_blank">andrew@cassidywebservices.co.uk</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I&#39;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" target="_blank">http://goo.gl/u8GWl</a><div>

<br><div class="gmail_quote"><div><div class="h5">On 2 January 2013 12:19, Cal Leeming [Simplicity Media Ltd] <span dir="ltr">&lt;<a href="mailto:cal.leeming@simplicitymedialtd.co.uk" target="_blank">cal.leeming@simplicitymedialtd.co.uk</a>&gt;</span> wrote:<br>

</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">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><font color="#888888"><div><br>Cal</div>
</font></span><br></div></div>_________________________________________________________________________<br>
Professional FreeSWITCH Consulting Services:<br>
<a href="mailto:consulting@freeswitch.org" target="_blank">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" target="_blank">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><span class="HOEnZb"><font color="#888888"><br><br clear="all"><div><br></div>-- <br><b>Andrew Cassidy BSc (Hons) MBCS SSCA</b><div>Managing Director<div><div><img><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>
</font></span></div>
<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></div>