On Mon, Aug 13, 2012 at 4:21 AM, Sonny Karlsson <[email protected]> wrote: > On Mon, Aug 13, 2012 at 12:52:48PM +0300, Lauri Kasanen wrote: >> On Mon, 13 Aug 2012 12:31:08 +0300 >> Lauri Kasanen <[email protected]> wrote: >> >> > On Sun, 12 Aug 2012 18:18:31 +0200 >> > Sonny Karlsson <[email protected]> wrote: >> > >> > > Current master causes build warnings when configured with --trace. It >> > > may be possible to solve this by changing the order of include >> > > statements or adding an extern declaration somewhere, but I have not >> > > found a way to do so yet. >> > >> > I'll take a look at building with trace today. But not inlining this >> > function is bad, as it's called extremely often (~9 times per request in >> > my test - for 200k requests I had 1.8M calls to this). >> > >> > So NAK this patch from me. >> >> Here's a patch for building with --trace enabled. Please test whether it >> works for you. >> >> - Lauri > > This patch works for me. Great work finding the problem!
the right solution, i did not catch this patch as i was reading the pending queue from bottom :) applied, thanks! > > -- > Sonny Karlsson > _______________________________________________ > Monkey mailing list > [email protected] > http://lists.monkey-project.com/listinfo/monkey -- Eduardo Silva http://edsiper.linuxchile.cl http://www.monkey-project.com _______________________________________________ Monkey mailing list [email protected] http://lists.monkey-project.com/listinfo/monkey
