<div dir="ltr"><div class="gmail_extra"><br>2014-09-19 19:34 GMT+08:00 Huw Davies <<a href="mailto:huw@codeweavers.com" target="_blank">huw@codeweavers.com</a>>:<br>><br>> On Thu, Sep 18, 2014 at 09:15:11PM +0800, Jactry Zeng wrote:<br>> > +static IRichEditOleImpl *get_reOle(ITextFontImpl *txtFont)<br>> > +{<br>> > +  if (txtFont->txtRge)<br>> > +    return txtFont->txtRge->reOle;<br>> > +  else<br>> > +    return txtFont->txtSel->reOle;<br>> > +}<br>><br>> You're using two-space indents here, while the rest is four-space.<br>> A couple of these have sneaked in in earlier patches too - make<br>> sure your editor is set up correctly!<br><br>So sorry, because sometime two-space indent was needed in other files </div><div class="gmail_extra">and my editor using two-space default. I will be more carefully next time!<br>Thanks for your review.<br clear="all"><div><br></div>-- <br><div dir="ltr"><div>Regards,<br></div><div>Jactry Zeng</div><div><br></div></div>
</div></div>