Hi Ran On Tue, Oct 16, 2012 at 9:58 PM, Ran Benita <ran...@gmail.com> wrote: > On Tue, Oct 16, 2012 at 04:05:33PM +0200, David Herrmann wrote: >> From: Ran Benita <ran...@gmail.com> >> >> This removes the complicated and undocumented hash-table creation-helper >> and replaces it with an autogenerated sorted array. The search uses simple >> bsearch() now. >> >> We also tried using gperf but it turned out to generate way to big >> hashtables and when reducing the size it isn't really faster than >> bsearch() anymore. >> >> There are no users complaining about the speed of keysym lookups and we >> have no benchmarks that tell that we are horribly slow. Hence, we can >> safely use the simpler approach and drop all that old code. > > I took the patches to my tree, with a couple of small non-functional > changes. It should appear in master in some time, unless Daniel has a > problem with breaking this particular API now. Though for me it seems > like a good time to drop all of our compat shims before releasing, as > the toolkits need to be updated for Wayland anyway. > > This turned out nicely, thanks again. > Ran
"...and test a couple 'thorny' cases..." haha, hilarious :) Thanks for picking it up. Daniel already mentioned that he is ok with the API-break so I think he will pull it right away from your tree. Regards David _______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/wayland-devel
