<div dir="ltr"><div dir="ltr">On Thu, 7 Nov 2019 at 15:24, Joshua C. Colp <<a href="mailto:jcolp@sangoma.com">jcolp@sangoma.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">On Thu, Nov 7, 2019 at 11:14 AM Steve Davies <<a href="mailto:davies147@gmail.com" target="_blank">davies147@gmail.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Hi,</div><div><br></div>So I've chosen to post this in -dev in case someone manages to come up with a pointer to an existing patch, because if not I imagine I will do the work and submit it myself. Advice, corrections and opinions are most welcome.<div><br></div><div>I noticed that 'res_monitor' is flagged as deprecated on the basis that 'app_mixmonitor' supports all the same features, and this led me down a rat-hole which I hope to dig my way out of...</div><div><br></div><div>- I need to record in- and out- channels separately so I can mix to a stereo file (YES)</div><div>- I need to pause and un-pause (NO)</div><div>  - Solution: do a stop and start/append instead (YES)</div><div>  - I only want to mix at the very end of the bridge/call (NO)</div><div>- The automon feature should also be deprecated or merged with automixmon (?)</div><div><br></div><div>I believe that the simplest solution to all of the above it to add a pause/unpause feature to app_mixmonitor, and extend 'struct ast_mixmonitor_methods' to allow broad support for an optional pause and unpause operation. This would then be implemented by simply pausing whether or not we write to the relevant filehandles. The audiohook itself would continue until stopped.</div><div><br></div><div>Perhaps there is good reason why this has not been done before? If so, please let me know before I go too far :)<br></div></div></blockquote><div><br></div><div>I'm not aware of any real reasons why it hasn't been done, except noone taking the time to do so or running into that functionality or getting around it in a different way.</div><div><br></div><div>I think deprecating automon is fine, although we don't really have a good ability to deprecate such things except to state it. They don't have support levels and other things like modules. </div></div><div><br></div></div></blockquote><div> </div><div>Thanks for the rapid feedback - I think I'll leave the automon vs automixmon deprecation alone for now then :) I'm guessing that res_monitor is not going to be removed any time soon based on your comments.</div><div><br></div><div>I will on the other hand have a crack at Pause/Unpause since when googling it the question has been asked quite a few times.</div><div><br></div><div>Regards,</div><div>Steve</div><div><br></div></div></div>