On Tue, Apr 12, 2011 at 3:54 PM, Kevin Pouget wrote: > > On Tue, Apr 12, 2011 at 9:41 AM, Pedro Alves wrote: > > Small nit I should have caught before: > > > > On Tuesday 12 April 2011 14:27:49, Kevin Pouget wrote: > >> +static int > >> +inf_has_multiple_thread_cb (struct thread_info *tp, void *data) > >> +{ > >> + int *has_multiple_threads = (int *) data; > > > > can you rename this local as well, in line with the other > > local in the other function? To "count", or "count_p", or something > > like that. Thanks. (pre-approved) > > fixed with count_p, thanks Hello, my copyright assignment is now ready, so I regenerated the patch against the up-to-date trunk Cordially, Kevin -- 2011-08-31 Kevin Pouget PR threads/12628 * linux-fork.c (checkpoint_command): Disallow checkpointing of processes with multiple threads. (inf_has_multiple_thread_cb): New function. (inf_has_multiple_threads): New function.