On 03/11/2011 12:01 PM, Ivana Hutarova Varekova wrote:
> put the usage output to separate function
> 
> 
> Signed-off-by: Ivana Hutarova Varekova <varek...@redhat.com>
Acked-By: Jan Safranek <jsafr...@redhat.com>

> ---
> 
>  src/tools/cgclassify.c |   21 ++++++++++++++++-----
>  1 files changed, 16 insertions(+), 5 deletions(-)
> 
> diff --git a/src/tools/cgclassify.c b/src/tools/cgclassify.c
> index cdd6a9d..c27249f 100644
> --- a/src/tools/cgclassify.c
> +++ b/src/tools/cgclassify.c
> @@ -31,6 +31,20 @@
>  
>  #define TEMP_BUF     81
>  
> +static void usage(int status, const char *program_name)
> +{
> +     if (status != 0) {
> +             fprintf(stderr, "Wrong input parameters,"
> +                     " try %s -h' for more information.\n",
> +                     program_name);
> +     } else {
> +             printf("usage is %s "
> +                     "[-g <list of controllers>:<relative path to cgroup>] "
> +                     "[--sticky | --cancel-sticky] <list of pids>  \n",
> +                     program_name);
> +     }
> +}
> +
>  /*
>   * Change process group as specified on command line.
>   */
> @@ -110,10 +124,7 @@ int main(int argc, char *argv[])
>  
>  
>       if (argc < 2) {
> -             fprintf(stderr, "usage is %s "
> -                     "[-g <list of controllers>:<relative path to cgroup>] "
> -                     "[--sticky | --cancel-sticky] <list of pids>  \n",
> -                     argv[0]);
> +             usage(1, argv[0]);
>               exit(2);
>       }
>  
> @@ -137,7 +148,7 @@ int main(int argc, char *argv[])
>                       flag |= CGROUP_DAEMON_CANCEL_UNCHANGE_PROCESS;
>                       break;
>               default:
> -                     fprintf(stderr, "Invalid command line option\n");
> +                     usage(1, argv[0]);
>                       exit(2);
>                       break;
>               }
> 
> 
> ------------------------------------------------------------------------------
> Colocation vs. Managed Hosting
> A question and answer guide to determining the best fit
> for your organization - today and in the future.
> http://p.sf.net/sfu/internap-sfd2d
> _______________________________________________
> Libcg-devel mailing list
> Libcg-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/libcg-devel


------------------------------------------------------------------------------
Colocation vs. Managed Hosting
A question and answer guide to determining the best fit
for your organization - today and in the future.
http://p.sf.net/sfu/internap-sfd2d
_______________________________________________
Libcg-devel mailing list
Libcg-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libcg-devel

Reply via email to