On Wed, 29 Jan 2020 12:59:27 -0600 Tom Zanussi <zanussi@xxxxxxxxxx> wrote: > +static struct synth_field *find_synth_field(struct synth_event *event, > + const char *field_name) > +{ > + struct synth_field *field = NULL; > + unsigned int i; > + > + for (i = 0; i < event->n_fields; i++) { > + field = event->fields[i]; > + if (strcmp(field->name, field_name) == 0) > + return field; > + } > + > + return NULL; > +} Why duplicate all theses functions and not use them in the synth_event_trace() directly? -- Steve