<div dir="ltr">Hi, James, I have created a PR <a href="https://github.com/xapian/xapian/pull/153">https://github.com/xapian/xapian/pull/153 </a><div><br></div><div>looking forward to the response.</div><div><br></div><div>Thanks</div><div class="gmail_extra"><br><div class="gmail_quote">2017-04-24 0:26 GMT+08:00 James Aylett <span dir="ltr"><<a href="mailto:james@tartarus.org" target="_blank">james@tartarus.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 23 Apr 2017, at 17:22, YuLun Cai <<a href="mailto:buptcyl@gmail.com">buptcyl@gmail.com</a>> wrote:<br>
<br>
> I'd suggest to start with you just look at moving the libmagic check after<br>
> the filesize checks, so you don't need to get into whether libmagic or<br>
> the database check is cheaper on average.<br>
><br>
> hi, Olly, I have moved the libmagic check after the filesize check directly,<br>
><br>
> <a href="https://github.com/caiyulun/xapian/commit/3a97d9ee5397fa900a473aa9b3d8eeb720177a4e" rel="noreferrer" target="_blank">https://github.com/caiyulun/<wbr>xapian/commit/<wbr>3a97d9ee5397fa900a473aa9b3d8ee<wbr>b720177a4e</a><br>
><br>
> can you provide your comments on it and give some advice about the next steps?<br>
<br>
</span>Can you create a pull request from this? It provides easier tools for feeding back on proposed changes.<br>
<span class="HOEnZb"><font color="#888888"><br>
J<br>
<br>
--<br>
 James Aylett<br>
 <a href="http://devfort.com" rel="noreferrer" target="_blank">devfort.com</a> — <a href="http://spacelog.org" rel="noreferrer" target="_blank">spacelog.org</a> — <a href="http://tartarus.org/james/" rel="noreferrer" target="_blank">tartarus.org/james/</a><br>
<br>
</font></span></blockquote></div><br></div></div>