On Wed, Jun 29, 2016 at 07:09:00PM +0800, Fengguang Wu <[email protected]> 
wrote:
> > And do notice slightly increased total time:

Hmm, that looks somewhat inconclusive to me. Time for independent
verification:

I did some tests by printing fully random, or the same 5/50/500 bg colour
change followed by "some words here", repeatedly. the test was effectively:

   time ./rxvt -e sh -c 'cat /tmp/x; sleep 1'

http://ue.tst.eu/97ddec618dd0f436da3478fa6619bfdf.txt

The first test was done twice with two different binaries, to get a feeling
for the variance.

It seems hashing is detrimental to all cases, very much so in the fully
random test (where this is expected), but also slightly for the case where
it should trigger a lot (5 and 50 different colours).

Possibly things will be different when the cache is colder (either
direction), but as it seems, hashing does not particularly help.

So we decided to go withoiut hashing at all.

Patch will be applied very soon btw., thanks again.

-- 
                The choice of a       Deliantra, the free code+content MORPG
      -----==-     _GNU_              http://www.deliantra.net
      ----==-- _       generation
      ---==---(_)__  __ ____  __      Marc Lehmann
      --==---/ / _ \/ // /\ \/ /      [email protected]
      -=====/_/_//_/\_,_/ /_/\_\

_______________________________________________
rxvt-unicode mailing list
[email protected]
http://lists.schmorp.de/mailman/listinfo/rxvt-unicode

Reply via email to