https://bugzilla.redhat.com/show_bug.cgi?id=1564149
--- Comment #28 from Xavi Hernandez <jaher...@redhat.com> --- (In reply to Aravinda VK from comment #27) > I prefer Right(`AlignEscapedNewlines=Right`), since Patch will have minimum > number of lines change on modification. For example, > > #define gf_log(dom, levl, fmt...) do { \ > FMT_WARN (fmt); \ > _gf_log (dom, __FILE__, __FUNCTION__, __LINE__, \ > levl, ##fmt); \ > } while (0) > > If longest line is changed then patch diff shows all of them as changed > lines in case of `AlignEscapedNewLines=Left` That makes sense. It's ok for me. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=CsBYbRgu0h&a=cc_unsubscribe _______________________________________________ Gluster-infra mailing list Gluster-infra@gluster.org http://lists.gluster.org/mailman/listinfo/gluster-infra