johnzon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Romain Manni-Bucau <rmannibu...@gmail.com>
Subject Re: Some suggestions for Strings.java and JsonLongImpl.java
Date Sat, 28 Jun 2014 21:58:48 GMT
Ok,

thanks a lot for the jira + patch (I know it looks too much for 1 word
but makes obvious you contributed the patch ;)).

Waiting to read the result of your tests ;)



Romain Manni-Bucau
Twitter: @rmannibucau
Blog: http://rmannibucau.wordpress.com/
LinkedIn: http://fr.linkedin.com/in/rmannibucau
Github: https://github.com/rmannibucau


2014-06-28 23:36 GMT+02:00 Hendrik Dev <hendrikdev22@gmail.com>:
> jira issue opened
>
> regarding the figures: i have some measurements on my own and the code
> shows a slight speedup, but i will look into
> https://github.com/RichardHightower/json-parsers-benchmark
>
> On Sat, Jun 28, 2014 at 11:16 PM, Romain Manni-Bucau
> <rmannibucau@gmail.com> wrote:
>> second fix is ok (please open a jira and attach a patch)
>>
>> the first one need figures (I used
>> https://github.com/RichardHightower/json-parsers-benchmark ), last
>> time I tried such a thing computing hashcode was too slow.
>>
>>
>> Romain Manni-Bucau
>> Twitter: @rmannibucau
>> Blog: http://rmannibucau.wordpress.com/
>> LinkedIn: http://fr.linkedin.com/in/rmannibucau
>> Github: https://github.com/rmannibucau
>>
>>
>> 2014-06-28 23:01 GMT+02:00 Hendrik Dev <hendrikdev22@gmail.com>:
>>> only constants in escape() method, unicode cache, clean up of possibly
>>> unnecessary code
>>> https://github.com/salyh/fleece_tmp/commit/a8d852bc3beeddc9607af1005a2cfdde2f252230
>>>
>>> long is integral
>>> https://github.com/salyh/fleece_tmp/commit/03dc2df406d171ded8db5199d4618786d29e5298
>>>
>>> Thanks
>>> Hendrik

Mime
View raw message