<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><span style="font-family:arial,sans-serif">On Wed, Apr 6, 2016 at 11:01 PM, Dave Chinner </span><span dir="ltr" style="font-family:arial,sans-serif"><<a href="mailto:david@fromorbit.com" target="_blank">david@fromorbit.com</a>></span><span style="font-family:arial,sans-serif"> wrote:</span><br></div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Apr 06, 2016 at 11:12:21AM +0200, Jan Tulak wrote:<br>
> On Fri, Apr 1, 2016 at 4:05 AM, Eric Sandeen <<a href="mailto:sandeen@sandeen.net">sandeen@sandeen.net</a>> wrote:<br>
> > On 3/24/16 6:15 AM, <a href="mailto:jtulak@redhat.com">jtulak@redhat.com</a> wrote:<br>
</span><span class="">> >                 if (sb_feat.finobt && sb_feat.finobtflag) {<br>
> >                         fprintf(stderr,<br>
> > _("warning: finobt not supported without CRC support, disabled.\n"));<br>
> >                 }<br>
> >                 sb_feat.finobt = 0;<br>
> ><br>
> > would suffice as before, no? Meh.  Not a big deal I guess....<br>
> ><br>
><br>
> ​Changed. Honestly, I don't like the strings starting at the beginning of<br>
> the line, because it breaks the indentation flow, but the rest of the code<br>
> uses this style, so I should stick to it.​<br>
<br>
</span>There's good reason for doing this - it makes it easy to grep the<br>
source code for a specific error that has been emitted. Indentation<br>
is useful for demonstrating logic flow, but it's harmful when it<br>
results in strings you might want to find being split up over<br>
multiple lines.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">​Mmm, yeah, a valid point. It is caused by the 80 chars limit, but there are reasons for that too​...</div></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> Thank you for the review. I will wait a little longer if someone spots<br>
> something more, before sending an updated patchset.​ :-)<br>
<br>
</span>Just send it - I almost got to pulling in this version and<br>
fixing the various comments directly myself yesterday....<br></blockquote><div><br></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">​OK. I will fix what Eric submitted in the mean time, test it to be sure I didn't broke anything and send.</div></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline"><br></div></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">Cheers,</div></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">Jan</div></div></div><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Jan Tulak<br></div><a href="mailto:jtulak@redhat.com" target="_blank">jtulak@redhat.com</a> / <a href="mailto:jan@tulak.me" target="_blank">jan@tulak.me</a></div></div></div></div>
</div></div>