----- Original Message ----- > From: "Dhaval Giani" <dhaval.gi...@gmail.com> > To: "Ivana Hutarova Varekova" <varek...@redhat.com> > Cc: "Libcg Development list" <libcg-devel@lists.sourceforge.net> > Sent: Tuesday, September 24, 2013 4:18:31 AM > Subject: Re: [Libcg-devel] [PATCH 15/19] cgsnapshot: unify help output > > On Tue, Sep 17, 2013 at 2:52 AM, Ivana Hutarova Varekova > <varek...@redhat.com> wrote: > > cgsnapshot: unify help output > > * use tabulators instead of spaces > > I am not sure what tabulators mean. Could you please clarify? (Sorry > for my ignorance)
Sorry for the mesunderstanding. I mean tab keys here. > > > * change formulation > > * change fprintf(stdout,...) to printf() > > > > Not sure why this change is needed. * add tab instead of spaces -> it unifies the help output indentation * add the formulation here "Generate the configuration file from the given controllers of control groups" was changed to "Generate the configuration file for given controllers" which sounds better for me. If you think the first formulation is better. I will change the patch and use the first formulation. * add fprintf stuff - it improves code readability, there is no need to have "fprintf(stdout,..." instead of "printf(..." Thanks Ivana > > Dhaval > ------------------------------------------------------------------------------ October Webinars: Code for Performance Free Intel webinars can help you accelerate application performance. Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from the latest Intel processors and coprocessors. See abstracts and register > http://pubads.g.doubleclick.net/gampad/clk?id=60133471&iu=/4140/ostg.clktrk _______________________________________________ Libcg-devel mailing list Libcg-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/libcg-devel
