FS-5832  regression.... After this change the sticky completes were deleting.  This should correct it

We can't use their new mb_gets() because they call exit(3) in the
event of a read error, which isn't what we want.

ref: http://my-basic.googlecode.com/svn/trunk@57

...after dos2unix conversion.

Having an unaltered base point in tree makes future merges with
upstream much easier.

ref: http://my-basic.googlecode.com/svn/trunk@54

gets(3) is going away; it's been removed by C11, and gcc now warns on
it, breaking the build.

fix mod_basic clang build warning, we don't need to declare var we don't use on this side of the ifdef

This commit also reverts 2 previous attempts to fix this very rare race issue spanning back to 2009

62ce8538 Patch from MOC
3a85348c FS-2302 mutex added around switch_xml_toxml()

The real problem was switch_xml_toxml_buf() was actually temporarily modifying the xml structure being searialized to make it appaer to be a root structure then serializing it and restoring the pointers.  This caused a non-threadsafe operation when some other thread was scanning the same xml structure.

This patch removes the modification and instead passes a new arg to switch_xml_toxml_r indicating to treat the structure as if it were a root structure.

This bug has been present since the induction of xml into FS.

We already had uuid_answer and uuid_pre_answer, so we might as well
add uuid_ring_ready.