[Devel] Re: [PATCH 2/4] Cleanup pre_restore_task comments

Oren Laadan orenl at librato.com
Wed Oct 14 15:34:05 PDT 2009


Ok, pulled.

Matt Helsley wrote:
> We don't need to include the name of the function in the comment unless
> it's a kerneldoc comment -- it's redundant. Also the comment before
> post_restore_task() was copy-pasted without modification.
> 
> Signed-off-by: Matt Helsley <matthltc at us.ibm.com>
> ---
>  checkpoint/process.c |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
> 
> diff --git a/checkpoint/process.c b/checkpoint/process.c
> index c51e036..17c08e8 100644
> --- a/checkpoint/process.c
> +++ b/checkpoint/process.c
> @@ -814,7 +814,7 @@ static int restore_task_pgid(struct ckpt_ctx *ctx)
>  	return ret;
>  }
>  
> -/* pre_restore_task - prepare the task for restore */
> +/* prepare the task for restore */
>  int pre_restore_task(struct ckpt_ctx *ctx)
>  {
>  	sigset_t sigset;
> @@ -844,7 +844,6 @@ int pre_restore_task(struct ckpt_ctx *ctx)
>  	return 0;
>  }
>  
> -/* pre_restore_task - prepare the task for restore */
>  void post_restore_task(struct ckpt_ctx *ctx)
>  {
>  	/* can happen if restart failed early */
_______________________________________________
Containers mailing list
Containers at lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers




More information about the Devel mailing list