FS-7117 #comment revert bf5210bf and implement it in ws.c please be sure to learn to use git commit hooks to properly associate commits with jiras

The second hunk in this patch isn't right.  In the past,
if tmp was null, it would not pass the if.

vid screen share placeholder --args --enable-usermedia-screen-capturing --usermedia-screen-capturing

this is why we can't have nice things.... C'mon chrome and mozilla its not that hard to both do the same spec

* commit '165f5421':
  mod_sofia: Set sip_to_tag on ringing indication for inbound channels.

When bridging a call, the to-tag used in the outgoing 180 Ringing
message for the inbound channel is unavailable until the channel has
been answered. For the outgoing channel this value is already available
through the sip_to_tag variable via the event socket.

This is solved this by setting sip_to_tag to the local leg's tag when
receiving a ringing indication for inbound channels. This will also make
the variable available in the CHANNEL_PROGRESS event through event
socket.

FS-7137 #resolve

Setting the 240k hard limit here is too aggressive and causes FS to
crash on startup.

Previously data returned from `memcache get` would be truncated at the
first NULL byte.

By using raw_write_function here to stream the returned memcache
value, we allow mod_memcache to be used for audio and other arbitrary
binary data.

Dave has a format module planned that relies on this.

Thanks-to: Dave Olszewski <dolszewski@marchex.com>

FS-7114 #resolve

ClangFormat is an attempt to build something like gofmt for C.

ref: http://clang.llvm.org/docs/ClangFormat.html
ref: http://clang.llvm.org/docs/ClangFormatStyleOptions.html

The clamp(v,a,b) function wraps v around the interval [a,b).

However prior to this commit, `clamp` was ignoring the third argument
and using the second argument again in its place.  This resulted in a
division by zero.  Hence `clamp` didn't work at all.

Even if the arguments were treated correctly, `clamp` incorrectly
multiplied rather than added whenever v < a.  This would have produced
bogus results.  (Thanks to Shona McNeill for pointing this out.)

Note that as implemented, `clamp` is undefined for b >= a.

These errors are present in the last upstream C version, v2.7.
They've been corrected in the C++ version that upstream now maintains
instead.

Thanks-to: Shona McNeill <prufrax@googlemail.com>

FS-7070 #resolve

With this commit we allow getting `make -jN` behavior by setting
`parallel=N` in `DEB_BUILD_OPTIONS`.  Setting `parallel=` will result
in `make -j` being run (i.e. no limit on the number of jobs).  The
default is to run one job at a time; this is still the safest bet when
building all modules.

Thanks to Peter Wu for suggesting a different way to do this.

ref: https://www.debian.org/doc/debian-policy/ch-source.html#s-debianrules-options