Em Thu, Oct 17, 2019 at 04:49:08PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Thu, Oct 17, 2019 at 03:37:33PM -0400, Steven Rostedt escreveu: > > On Thu, 17 Oct 2019 16:28:32 -0300 > > Arnaldo Carvalho de Melo <[email protected]> wrote: > > > > > Em Thu, Oct 17, 2019 at 02:41:14PM -0400, Steven Rostedt escreveu: > > > > On Thu, 17 Oct 2019 14:38:41 -0400 > > > > Steven Rostedt <[email protected]> wrote: > > > > > > > > > struct tep_event *trace_find_next_event(struct tep_handle *pevent, > > > > > struct tep_event *event) > > > > > { > > > > > + static struct tep_event **all_events; > > > > > static int idx; > > > > > int events_count; > > > > > - struct tep_event *all_events; > > > > > > > > If we are going to use static variables, let's make them all static and > > > > optimize it a little more... > > > > > > I'll test it, but can't you have this somewhere else, i.e. at > > > tep_handle perhaps? > > > > > > > > > > Or we can nuke the function entirely, it's a rather silly helper > > anyway. Just do this: > > I like it, that is a good nuke use, nice button to press! :-) > > Testing it now...
Reported-by: Daniel Bristot de Oliveira <[email protected]> Tested-by: Arnaldo Carvalho de Melo <[email protected]> Works like a charm, thanks! - Arnaldo
