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

Reply via email to