<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">such things would typically be submitted either via a pull request on our github respository, or a patch to ardour-dev.<br><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">often we would look at and act on them in a day or three; at the current time, we're still fairly deeply immersed in architectural work, so they would take much longer to be evaluated.<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 2, 2018 at 9:22 AM, Chris Caudle <span dir="ltr"><<a href="mailto:chris@chriscaudle.org" target="_blank">chris@chriscaudle.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, March 2, 2018 8:11 am, Ben Loftis wrote:<br>
> ....more ideas welcome!<br>
<br>
Is there any interest in minor changes that (probably) don't have any<br>
effect on functionality?<br>
I see pages of warnings like this when building:<br>
../gtk2_ardour/ardour_ui_<wbr>dialogs.cc:439:33: warning: missed loop<br>
optimization, the loop counter may overflow [-Wunsafe-loop-optimizations]<br>
  for (i = candidates.begin(); i != candidates.end(); ++i) {<br>
                               ~~^~~~~~~~~~~~~~~~~~~<br>
<br>
<br>
Not a big deal, but at one point I was habituated to make sure code<br>
compiled without warnings both to make sure any real logic errors did not<br>
slip through, and also to make sure the compiler was able to optimize<br>
fully.  In this case the warning is about a missed optimization, I assume<br>
that the counter isn't really going to overflow, or if it does wrap around<br>
and count twice it won't hurt anything, but I haven't looked at that<br>
section in enough detail to make sure there are no corner cases.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Chris Caudle<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
______________________________<wbr>_________________<br>
Ardour-Users mailing list<br>
<a href="mailto:ardour-users@lists.ardour.org">ardour-users@lists.ardour.org</a><br>
To unsubscribe  or change your mailing preferences please visit:<br>
<a href="http://lists.ardour.org/listinfo.cgi/ardour-users-ardour.org" rel="noreferrer" target="_blank">http://lists.ardour.org/<wbr>listinfo.cgi/ardour-users-<wbr>ardour.org</a><br>
</div></div></blockquote></div><br></div>