Re: [Xen-devel] [PATCH] Add clang-format file for Xen Hypervisor format

2017-04-18 Thread Lars Kurth
Ishani, > On 12 Apr 2017, at 16:38, Ishani wrote: > > Hello, > > I seemed to have overwritten the correct file by the output of the > clang-format tool. My main idea is to compare output file and correct file > and take their comparison. Since there are features which are partially > impleme

Re: [Xen-devel] [PATCH] Add clang-format file for Xen Hypervisor format

2017-04-12 Thread Ishani
hani Chugh" Cc: "cardoe" , "Lars Kurth" , "xen-devel" Sent: Wednesday, April 12, 2017 8:46:48 PM Subject: Re: [Xen-devel] [PATCH] Add clang-format file for Xen Hypervisor format >>> On 12.04.17 at 12:49, wrote: > --- /dev/null > +++ b/tools/clang-f

Re: [Xen-devel] [PATCH] Add clang-format file for Xen Hypervisor format

2017-04-12 Thread Jan Beulich
>>> On 12.04.17 at 12:49, wrote: > --- /dev/null > +++ b/tools/clang-format/tests/correct.c Is this file supposed to be in correct Xen style? Its name suggests so, but its contents don't. Jan ___ Xen-devel mailing list Xen-devel@lists.xen.org https:/

Re: [Xen-devel] [PATCH] Add clang-format file for Xen Hypervisor format

2017-04-12 Thread Lars Kurth
> On 12 Apr 2017, at 12:51, Ishani wrote: > > Hello, > > Thanks for the comments. > I have created a markdown file for clang-format doc which I will add. > >> If so, you want to include this information in the cover letter (see above). >> Or you >could write little bash script which shows ho

Re: [Xen-devel] [PATCH] Add clang-format file for Xen Hypervisor format

2017-04-12 Thread Ishani
Hello, Thanks for the comments. I have created a markdown file for clang-format doc which I will add. >If so, you want to include this information in the cover letter (see above). >Or you >could write little bash script which shows how this works Can you elaborate on the task of bash script? S

Re: [Xen-devel] [PATCH] Add clang-format file for Xen Hypervisor format

2017-04-12 Thread Lars Kurth
Hi Ishani, > On 12 Apr 2017, at 11:49, Ishani Chugh > wrote: Here you would normally add a short description of the patch (also called cover letter). In this case, you probably want to describe: a) What you have done b) Link to https://wiki.xenproject.org/wiki/Outreach_Program_Projects#Code_S

[Xen-devel] [PATCH] Add clang-format file for Xen Hypervisor format

2017-04-12 Thread Ishani Chugh
Signed-off-by: Ishani Chugh --- tools/clang-format/tests/.clang-format | 88 ++ tools/clang-format/tests/correct.c | 73 tools/clang-format/tests/test.c| 69 ++ 3 files changed, 230 insertions(+) cre