[patch] skiplist opt

0 views
Skip to first unread message

Mike Leddy

unread,
Apr 28, 2008, 11:43:41 PM4/28/08
to StrokeDB
Just starting to get interested and I'm reading the code. I couldn't
help noticing a straightforward
optimization whilst reading skiplist.rb. I know that optimizations
aren't a priority at the moment,
but if it doesn't interfere with the readability consider accepting
it.

Thanks for an interesting project..... lots of great code to read.

Regards,

Mike

PS its seems i can't attach a file - uploaded to
strokedb_skiplist_opt.patch in Files


Oleg Andreev

unread,
Apr 29, 2008, 12:41:39 AM4/29/08
to stro...@googlegroups.com

On 29.04.2008, at 7:43, Mike Leddy wrote:

>
> Just starting to get interested and I'm reading the code. I couldn't
> help noticing a straightforward
> optimization whilst reading skiplist.rb. I know that optimizations
> aren't a priority at the moment,
> but if it doesn't interfere with the readability consider accepting
> it.

Hi!

Optimizations are always welcome. Especially in a Skiplist.
We are going to replace current Skiplist class with a SimpleSkiplist
soon, which has several optimizations already and should be optimized
further.

Anyway, thanks for your attention!
Oleg.


Reply all
Reply to author
Forward
0 new messages