<br clear="all">片云天共远永夜月同孤<br>
<br><br><div class="gmail_quote">---------- Forwarded message ----------<br>From: <b class="gmail_sendername">Xiangjun Wu</b> <span dir="ltr"><<a href="mailto:netawater@gmail.com">netawater@gmail.com</a>></span><br>
Date: Sun, Aug 16, 2009 at 10:18 PM<br>Subject: Re: An Elephant task<br>To: <a href="mailto:leslie.polzer@gmx.net">leslie.polzer@gmx.net</a><br><br><br>I have added the test case with his patch, please note the test data file is UTF-8 file.<br>
<br>In my test, GET-INSTACES-BY-RANG is OK, but GET-ININSTACE-BY-VALUE and GET-ININSTACES-BY-VALUE are NG when new code run in old store.<br>
<br>Please check it. <br><br>片云天共远永夜月同孤<div><div></div><div class="h5"><br>
<br><br><div class="gmail_quote">On Thu, Aug 13, 2009 at 4:44 AM, Leslie P. Polzer <span dir="ltr"><<a href="mailto:sky@viridian-project.de" target="_blank">sky@viridian-project.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

<div><br>
Xiangjun Wu wrote:<br>
> I have studied his patch and here is my conclusion:<br>
><br>
> libberkeley-db.c<br>
> -    return wcs_cmp((wchar_t*)ad+5+offset, read_int32(ad+offset, 1),<br>
> (wchar_t*)bd+5+offset, read_int32(bd+offset, 1));<br>
><br>
> +   return wcs_cmp((wchar_t*)(ad+5+offset), read_int32(ad+offset, 1),<br>
> (wchar_t*)(bd+5+offset), read_int32(bd+offset, 1));<br>
>>>>> It want to type cast, but I believe the compiler also do it.<br>
<br>
</div>I think gcc does, but the compiler is not required to upgrade/cast<br>
the type of the other arguments. The second form seems more concise<br>
so I guess it's better.<br>
<div><br>
<br>
> unicode.lisp<br>
> he modified serialize-to-utf16le, serialize-to-utf32le and<br>
> deserialize-string ((type (eql :utf16le)), deserialize-string ((type<br>
> (eql :utf32le))<br>
> with respect to serialize and deserialize utf16 or utf32 string. I'm<br>
> not sure which is big endian or little endian, however he only do a<br>
> revert operation, It is OK.<br>
<br>
</div>What about backwards compatibility? Say I've got a store with<br>
the old sorting order and load the code that uses the new one.<br>
<br>
Will the data sorted in the old way be read correctly?<br>
<div><br>
<br>
> his test file is OK which based the rightness of string< for utf16 and utf32<br>
> string.<br>
><br>
> BerkeleyDB-tests is OK after I apply his patch in a clean repository.<br>
<br>
</div>That's great! However we should also add the test cases he provided<br>
to the test suite. Can you do that?<br>
<br>
  Thanks a lot! :)<br>
<div><div></div><div><br>
    Leslie<br>
<br>
--<br>
<a href="http://www.linkedin.com/in/polzer" target="_blank">http://www.linkedin.com/in/polzer</a><br>
<br>
</div></div></blockquote></div><br>
</div></div></div><br>