Re: Ardour contributions ( was: Level needed ... )
Paul Davis <[email protected]>
| Newsgroups | gmane.comp.audio.ardour.user |
|---|---|
| Message-ID | <CAFa_cK=7CA91vsUt7HiJDMPEVB5a_gHzYrz9Jrdyq6AK5V+TZA@mail.gmail.com> |
such things would typically be submitted either via a pull request on our github respository, or a patch to ardour-dev. 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. On Fri, Mar 2, 2018 at 9:22 AM, Chris Caudle <[email protected]> wrote: > On Fri, March 2, 2018 8:11 am, Ben Loftis wrote: > > ....more ideas welcome! > > Is there any interest in minor changes that (probably) don't have any > effect on functionality? > I see pages of warnings like this when building: > ../gtk2_ardour/ardour_ui_dialogs.cc:439:33: warning: missed loop > optimization, the loop counter may overflow [-Wunsafe-loop-optimizations] > for (i = candidates.begin(); i != candidates.end(); ++i) { > ~~^~~~~~~~~~~~~~~~~~~ > > > Not a big deal, but at one point I was habituated to make sure code > compiled without warnings both to make sure any real logic errors did not > slip through, and also to make sure the compiler was able to optimize > fully. In this case the warning is about a missed optimization, I assume > that the counter isn't really going to overflow, or if it does wrap around > and count twice it won't hurt anything, but I haven't looked at that > section in enough detail to make sure there are no corner cases. > > -- > Chris Caudle > > > _______________________________________________ > Ardour-Users mailing list > [email protected] > To unsubscribe or change your mailing preferences please visit: > http://lists.ardour.org/listinfo.cgi/ardour-users-ardour.org > _______________________________________________ Ardour-Users mailing list [email protected] To unsubscribe or change your mailing preferences please visit: http://lists.ardour.org/listinfo.cgi/ardour-users-ardour.org