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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.