<span itemscope itemtype="http://schema.org/EmailMessage"><p itemprop="description">Planning to postpone this meeting, and idea is to work in more collaborated way off-line, instead of being in a meeting. we believe it would give everyone (those who didn&#39;t attend) too a fair chance to submit their opinion.<p>For now, we will continue with Nigel&#39;s clang-format repo for this to experiment with different options. [<a href="https://github.com/nigelbabu/clang-format-sample">https://github.com/nigelbabu/clang-format-sample</a>]<p>The plan on this is to go with a sample gluster file, which would have complex macros, a call with STACK_WIND/UNWIND function call. A switch case, a for loop, a do/while loop. Also a list_for_each loop. Have locked region. A sample 4-5 level depth if/else checks, etc. <p>With this sample file, having the .clang-format decided as either Chromium or Mozilla as a base (with IndentSize set to 4 space), would be a good start. We will also make sure to have  all the agreed points in bugzilla, and add it to clang-format file, and also regenerate the sample file. So, everyone gets an idea how the target file would look like. If everyone agrees, by the end of the week, we will have an agreement, so we can go ahead and make this possible before 4.1 release branching. (So, our backport efforts will be reduced drastically).<p>-Amar</p></span><span itemscope itemtype="http://schema.org/InformAction"><span style="display:none" itemprop="about" itemscope itemtype="http://schema.org/EmailMessage"><meta itemprop="description" content="atumball@redhat.com: Planning to postpone this meeting, and idea is to work in more collaborated way off-line, instead of being in a meeting. we believe it would give everyone (those who didn&#39;t attend) too a fair chance to submit their opinion. For now, we will continue with Nigel&#39;s clang-format repo for this to experiment with different options. [https://github.com/nigelbabu/clang-format-sample] The plan on this is to go with a sample gluster file, which would have complex macros, a call with STACK_WIND/UNWIND function call. A switch case, a for loop, a do/while loop. Also a list_for_each loop. Have locked region. A sample 4-5 level depth if/else checks, etc. With this sample file, having the .clang-format decided as either Chromium or Mozilla as a base (with IndentSize set to 4 space), would be a good start. We will also make sure to have all the agreed points in bugzilla, and add it to clang-format file, and also regenerate the sample file. So, everyone gets an idea how the target file would look like. If everyone agrees, by the end of the week, we will have an agreement, so we can go ahead and make this possible before 4.1 release branching. (So, our backport efforts will be reduced drastically). -Amar"/></span><span itemprop="object" itemscope itemtype="http://schema.org/Event"><div style=""><table cellspacing="0" cellpadding="8" border="0" summary="" style="width:100%;font-family:Arial,Sans-serif;border:1px Solid #ccc;border-width:1px 2px 2px 1px;background-color:#fff;"><tr><td><meta itemprop="eventStatus" content="http://schema.org/EventScheduled"/><div style="padding:2px"><span itemprop="publisher" itemscope itemtype="http://schema.org/Organization"><meta itemprop="name" content="Google Calendar"/></span><meta itemprop="eventId/googleCalendar" content="0lg6ib7qjkumf6ftr90lh337ap"/><h3 style="padding:0 0 6px 0;margin:0;font-family:Arial,Sans-serif;font-size:16px;font-weight:bold;color:#222"><span itemprop="name">Coding Standard: Automation</span></h3><div style="padding-bottom:15px;font-family:Arial,Sans-serif;font-size:13px;color:#222;white-space:pre-wrap!important;white-space:-moz-pre-wrap!important;white-space:-pre-wrap!important;white-space:-o-pre-wrap!important;white-space:pre;word-wrap:break-word"><span>BJ: <a href="https://www.google.com/url?q=https%3A%2F%2Fbluejeans.com%2F205933580&amp;sa=D&amp;ust=1524475718810000&amp;usg=AFQjCNGTaaNTzaWd-W-uynMJetu6B5iKrw" target="_blank">https://bluejeans.com/205933580</a><br><br>We will talk and come to agreement on <a href="https://www.google.com/url?q=https%3A%2F%2Fbugzilla.redhat.com%2Fshow_bug.cgi%3Fid%3D1564149&amp;sa=D&amp;ust=1524475718811000&amp;usg=AFQjCNFdx9TMiO-6fNfyyQFlI-P_1ewpHg" target="_blank">https://bugzilla.redhat.com/show_bug.cgi?id=1564149</a><br><br>It was agreed that we will go ahead with format change automation, so, goal of this meeting is to pick the right options.<br><br>Goal is to get gluster&#39;s own `.clang-format` file. Once that file is agreed upon, we will go ahead and create a job for fixing the patches for format, and also fix the codebase to get the formats.<br><br>Pre-work if you are interested, read about : <a href="https://www.google.com/url?q=https%3A%2F%2Fclang.llvm.org%2Fdocs%2FClangFormatStyleOptions.html&amp;sa=D&amp;ust=1524475718811000&amp;usg=AFQjCNFHjwiLZMlDA3zDhSI4kPiYozHRlg" target="_blank">https://clang.llvm.org/docs/ClangFormatStyleOptions.html</a><br><br>Also pick a gluster file which would pass through agreed format, so you can validate how it looks after formatting. Instead of waiting for this to happen, we can see is this good enough?<br><br>Few things we mostly agree:<br><br><pre> !AllowShortIfStatementsOnASingleLine !AllowShortLoopsOnASingleLine BraceWrapping(!AfterControlStatement) BraceWrapping(AfterFunction) BraceWrapping(!BeforeElse) ColumnLimit(80) IndentWidth(4) PointerAlignment(PAS_Right) SpaceBeforeParens(SBPO_Always) TabWidth(8) UseTab(UT_Never)</pre><br><pre><span>  BinPackParameters=true</span></pre><pre><font face="Roboto, Helvetica, Arial, sans-serif">  </font>AlignEscapedNewLinesLeft=false</pre><pre> AlignConsecutiveDeclarations=true</pre><pre><span>  AlignConsecutiveAssignments=true</span></pre><pre> AlwaysBreakAfterReturnType = true</pre><br><br>More options which we can discuss:<br><pre>!IndentCaseLabels<span>SpaceBeforeParens = ControlStatements</span> </pre><pre><br></pre><br>I propose two steps as preventing history:<br><br>* The commit before the mass-format-change commit will maintained as a separate branch. (No cost of space, but everyone clearly knows where to go for history, when git blame pointing to the commit of mass changes).<br>* Similarly, to get history of pre-2009 (currently &#39;historic&#39; repo), I personally feel moving  <a href="https://www.google.com/url?q=https%3A%2F%2Fgithub.com%2Famarts%2Fglusterfs%2Fcommits%2Fgit-based-history-from-historic&amp;sa=D&amp;ust=1524475718817000&amp;usg=AFQjCNFqMDCu47TjNU2abBOcA22hvqWInw" target="_blank">https://github.com/amarts/glusterfs/commits/git-based-history-from-historic</a>, as a separate branch in gluster/glusterfs would help. Again, today people has to switch repositories for this.</span><meta itemprop="description" content="BJ:&amp;nbsp;https://bluejeans.com/205933580We will talk and come to agreement on&amp;nbsp;https://bugzilla.redhat.com/show_bug.cgi?id=1564149It was agreed that we will go ahead with format change automation, so, goal of this meeting is to pick the right options.Goal is to get gluster&#39;s own `.clang-format` file. Once that file is agreed upon, we will go ahead and create a job for fixing the patches for format, and also fix the codebase to get the formats.Pre-work if you are interested, read about : https://clang.llvm.org/docs/ClangFormatStyleOptions.htmlAlso pick a gluster file which would pass through agreed format, so you can validate how it looks after formatting. Instead of waiting for this to happen, we can see is this good enough?Few things we mostly agree: !AllowShortIfStatementsOnASingleLine !AllowShortLoopsOnASingleLine BraceWrapping(!AfterControlStatement) BraceWrapping(AfterFunction) BraceWrapping(!BeforeElse) ColumnLimit(80) IndentWidth(4) PointerAlignment(PAS_Right) SpaceBeforeParens(SBPO_Always) TabWidth(8) UseTab(UT_Never)  BinPackParameters=true  AlignEscapedNewLinesLeft=false AlignConsecutiveDeclarations=true  AlignConsecutiveAssignments=true AlwaysBreakAfterReturnType = trueMore options which we can discuss:!IndentCaseLabelsSpaceBeforeParens = ControlStatements I propose two steps as preventing history:* The commit before the mass-format-change commit will maintained as a separate branch. (No cost of space, but everyone clearly knows where to go for history, when git blame pointing to the commit of mass changes).* Similarly, to get history of pre-2009 (currently &#39;historic&#39; repo), I personally feel moving&amp;nbsp; https://github.com/amarts/glusterfs/commits/git-based-history-from-historic, as a separate branch in gluster/glusterfs would help. Again, today people has to switch repositories for this."/></div><table cellpadding="0" cellspacing="0" border="0" summary="Event details"><tr><td style="padding:0 1em 10px 0;font-family:Arial,Sans-serif;font-size:13px;color:#888;white-space:nowrap" valign="top"><div><i style="font-style:normal">When</i></div></td><td style="padding-bottom:10px;font-family:Arial,Sans-serif;font-size:13px;color:#222" valign="top"><time itemprop="startDate" datetime="20180423T123000Z"></time><time itemprop="endDate" datetime="20180423T132000Z"></time>Mon Apr 23, 2018 6pm – 6:50pm <span style="color:#888">India Standard Time</span></td></tr><tr><td style="padding:0 1em 10px 0;font-family:Arial,Sans-serif;font-size:13px;color:#888;white-space:nowrap" valign="top"><div><i style="font-style:normal">Who</i></div></td><td style="padding-bottom:10px;font-family:Arial,Sans-serif;font-size:13px;color:#222" valign="top"><table cellspacing="0" cellpadding="0"><tr><td style="padding-right:10px;font-family:Arial,Sans-serif;font-size:13px;color:#222"><span style="font-family:Courier New,monospace">&#x2022;</span></td><td style="padding-right:10px;font-family:Arial,Sans-serif;font-size:13px;color:#222"><div><div style="margin:0 0 0.3em 0"><span itemprop="attendee" itemscope itemtype="http://schema.org/Person"><span itemprop="name" class="notranslate">atumball@redhat.com</span><meta itemprop="email" content="atumball@redhat.com"/></span><span itemprop="organizer" itemscope itemtype="http://schema.org/Person"><meta itemprop="name" content="atumball@redhat.com"/><meta itemprop="email" content="atumball@redhat.com"/></span><span style="font-size:11px;color:#888"> - organizer</span></div></div></td></tr><tr><td style="padding-right:10px;font-family:Arial,Sans-serif;font-size:13px;color:#222"><span style="font-family:Courier New,monospace">&#x2022;</span></td><td style="padding-right:10px;font-family:Arial,Sans-serif;font-size:13px;color:#222"><div><div style="margin:0 0 0.3em 0"><span itemprop="attendee" itemscope itemtype="http://schema.org/Person"><span itemprop="name" class="notranslate">jeff@pl.atyp.us</span><meta itemprop="email" content="jeff@pl.atyp.us"/></span></div></div></td></tr><tr><td style="padding-right:10px;font-family:Arial,Sans-serif;font-size:13px;color:#222"><span style="font-family:Courier New,monospace">&#x2022;</span></td><td style="padding-right:10px;font-family:Arial,Sans-serif;font-size:13px;color:#222"><div><div style="margin:0 0 0.3em 0"><span itemprop="attendee" itemscope itemtype="http://schema.org/Person"><span itemprop="name" class="notranslate">nbabu@redhat.com</span><meta itemprop="email" content="nigelb@redhat.com"/></span></div></div></td></tr><tr><td style="padding-right:10px;font-family:Arial,Sans-serif;font-size:13px;color:#222"><span style="font-family:Courier New,monospace">&#x2022;</span></td><td style="padding-right:10px;font-family:Arial,Sans-serif;font-size:13px;color:#222"><div><div style="margin:0 0 0.3em 0"><span itemprop="attendee" itemscope itemtype="http://schema.org/Person"><span itemprop="name" class="notranslate">srangana@redhat.com</span><meta itemprop="email" content="srangana@redhat.com"/></span></div></div></td></tr><tr><td style="padding-right:10px;font-family:Arial,Sans-serif;font-size:13px;color:#222"><span style="font-family:Courier New,monospace">&#x2022;</span></td><td style="padding-right:10px;font-family:Arial,Sans-serif;font-size:13px;color:#222"><div><div style="margin:0 0 0.3em 0"><span itemprop="attendee" itemscope itemtype="http://schema.org/Person"><span itemprop="name" class="notranslate">gluster-devel@gluster.org</span><meta itemprop="email" content="gluster-devel@gluster.org"/></span></div></div></td></tr><tr><td style="padding-right:10px;font-family:Arial,Sans-serif;font-size:13px;color:#222"><span style="font-family:Courier New,monospace">&#x2022;</span></td><td style="padding-right:10px;font-family:Arial,Sans-serif;font-size:13px;color:#222"><div><div style="margin:0 0 0.3em 0"><span itemprop="attendee" itemscope itemtype="http://schema.org/Person"><span itemprop="name" class="notranslate">jahernan@redhat.com</span><meta itemprop="email" content="jahernan@redhat.com"/></span></div></div></td></tr></table></td></tr></table></div></td></tr></table></div></span></span>