Hui Zhu wrote: > Thanks Michael. > > With change: > static struct target_ops record_ops; > @@ -338,7 +344,10 @@ record_list_release_following (struct re > { > rec = tmp->next; > if (record_entry_release (tmp) == record_end) > + { > record_insn_num--; > + record_insn_count--; > + } > tmp = rec; > } > } > @@ -367,7 +376,6 @@ record_list_release_first (void) > /* tmp is now isolated, and can be deleted. */ > if (record_entry_release (tmp) == record_end) > { > - record_insn_num--; > break; /* End loop at first record_end. */ > } > To: > static struct target_ops record_ops; > @@ -338,7 +344,10 @@ record_list_release_following (struct re > { > rec = tmp->next; > if (record_entry_release (tmp) == record_end) > + { > record_insn_num--; > + record_insn_count--; > + } > tmp = rec; > } > } > @@ -367,7 +376,6 @@ record_list_release_first (void) > /* tmp is now isolated, and can be deleted. */ > if (record_entry_release (tmp) == record_end) > - { > - record_insn_num--; > break; /* End loop at first record_end. */ > - } > > Just some format change. > > I am OK with this patch. Thanks, changes attached and committed.