Re: [Xen-devel] [PATCH v3 1/5] xl: convert main() exit codes to EXIT_[SUCCESS|FAILURE]

[Dario Faggioli <dario.faggioli@xxxxxxxxxx>]

On Tue, 2015-10-27 at 11:11 +0000, Wei Liu wrote:
> On Tue, Oct 27, 2015 at 03:20:48PM +0530, Harmandeep Kaur wrote:
> [...]
> >  };
> >  
> > +/*
> > + *The xl process should always return either EXIT_SUCCESS or
> > + *EXIT_FAILURE. main_* functions, implementing the various xl
> > + *commands, can be treated as main() as if they are returning
> > + *a process exit status and not a function return value.
> > + */
> > +
> 
> Missing spaces after "*". E.g.
>    /*
>     * The xl process ....
>     * EXIT_FAILURE
>     * ...
>     */
> 
> With that fixed:
> 
> Acked-by: Wei Liu <wei.liu2@xxxxxxxxxx>
> 
> (This means after fixing the minor problem you can put the above tag
> in
> your commit message. If in doubt, please ask.)
> 
And, for what is worth, this one as well:

Reviewed-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>

Thanks and Regards,
Dario
-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel