--------_3E56479AED4F017A0008_MULTIPART_MIXED_ Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit > Nice. Perhaps could be made more general by checking if the value is a > Fixnum first? (And if not, it could do h[x] [x].succ) generic version 'hashsuccg.c' here. $ ruby -r mkmf -e'create_makefile("hashsuccg")' $ make # make install Usage: require "hashsuccg" h ash.new("a") h.succ! "foo" # {"foo" b"} h.succ! "foo" # {"foo" c"} Benchmark: $ time ruby b_hashsucc.rb # original hashsucc.c ruby b_hashsucc.rb 5.04s user 0.01s system 100% cpu 5.048 total $ time ruby b_hashsuccg.b # generic version ruby b_hashsuccg.rb 6.23s user 0.01s system 100% cpu 6.236 total 'hashsucc.c' bug-fix version attached. -- MoonWolf <moonwolf / moonwolf.com> --------_3E56479AED4F017A0008_MULTIPART_MIXED_ Content-Type: application/octet-stream; name="hashsucc.c" Content-Disposition: attachment; filename ashsucc.c" Content-Transfer-Encoding: base64 I2luY2x1ZGUgInJ1YnkuaCIKI2luY2x1ZGUgInN0LmgiCgpWQUxVRQpoYXNoX3N1Y2MoaGFzaCwg a2V5KQogICAgVkFMVUUgaGFzaCwga2V5Owp7CiAgVkFMVUUgYXJ5LCB2YWwsIG9uZTsKICBzdF90 YWJsZSAqdGJsOwogIGludCBsZW5ndGgsIGNvdW50LCBvZnM7CiAgCiAgb25lID0gSU5UMkZJWCgx KTsKICB0YmwgPSBSSEFTSChoYXNoKS0+dGJsOwogIAogIHN3aXRjaChUWVBFKGtleSkpIHsKICBj YXNlIFRfU1RSSU5HOgogICAgaWYgKHN0X2xvb2t1cCh0YmwsIGtleSwgJnZhbCkpIHsKICAgICAg Y291bnQgPSBGSVgySU5UKHZhbCk7CiAgICAgIGNvdW50Kys7CiAgICAgIHN0X2luc2VydCh0Ymws IGtleSwgSU5UMkZJWChjb3VudCkpOwogICAgfSBlbHNlIHsKICAgICAgc3RfYWRkX2RpcmVjdCh0 YmwsIHJiX3N0cl9uZXc0KGtleSksIG9uZSk7CiAgICB9CiAgICBicmVhazsKICBjYXNlIFRfQVJS QVk6CiAgICBhcnkgPSBrZXk7CiAgICBsZW5ndGggPSBSQVJSQVkoYXJ5KS0+bGVuOwogICAgZm9y KG9mcz0wOyBvZnMgPCBsZW5ndGg7IG9mcysrKSB7CiAgICAgIGtleSA9IFJBUlJBWShhcnkpLT5w dHJbb2ZzXTsKICAgICAgaWYgKHN0X2xvb2t1cCh0YmwsIGtleSwgJnZhbCkpIHsKICAgICAgICBj b3VudCA9IEZJWDJJTlQodmFsKTsKICAgICAgICBjb3VudCsrOwogICAgICAgIHN0X2luc2VydCh0 YmwsIGtleSwgSU5UMkZJWChjb3VudCkpOwogICAgICB9IGVsc2UgewogICAgICAgIHN0X2FkZF9k aXJlY3QodGJsLCByYl9zdHJfbmV3NChrZXkpLCBvbmUpOwogICAgICB9CiAgICB9CiAgICBicmVh azsKICB9CiAgcmV0dXJuIGhhc2g7Cn0KCnZvaWQKSW5pdF9oYXNoc3VjYygpCnsKICByYl9kZWZp bmVfbWV0aG9kKHJiX2NIYXNoLCJzdWNjISIsIGhhc2hfc3VjYywgMSk7Cn0K --------_3E56479AED4F017A0008_MULTIPART_MIXED_ Content-Type: application/octet-stream; name="hashsuccg.c" Content-Disposition: attachment; filename ashsuccg.c" Content-Transfer-Encoding: base64 I2luY2x1ZGUgInJ1YnkuaCIKI2luY2x1ZGUgInN0LmgiCgpzdGF0aWMgSUQgaWRfc3VjYywgaWRf ZGVmYXVsdDsKCgpWQUxVRQpoYXNoX3N1Y2MoaGFzaCwga2V5KQogICAgVkFMVUUgaGFzaCwga2V5 Owp7CiAgVkFMVUUgYXJ5LCB2YWw7CiAgc3RfdGFibGUgKnRibDsKICBpbnQgbGVuZ3RoLCBjb3Vu dCwgb2ZzOwogIAogIHRibCA9IFJIQVNIKGhhc2gpLT50Ymw7CiAgCiAgc3dpdGNoKFRZUEUoa2V5 KSkgewogIGNhc2UgVF9TVFJJTkc6CiAgICBpZiAoc3RfbG9va3VwKHRibCwga2V5LCAmdmFsKSkg ewogICAgICB2YWwgPSByYl9mdW5jYWxsKHZhbCwgaWRfc3VjYywgMCk7CiAgICAgIHN0X2luc2Vy dCh0YmwsIGtleSwgdmFsKTsKICAgIH0gZWxzZSB7CiAgICAgIHZhbCA9IHJiX2Z1bmNhbGwoaGFz aCwgaWRfZGVmYXVsdCwgMCk7CiAgICAgIHZhbCA9IHJiX2Z1bmNhbGwodmFsLCBpZF9zdWNjLCAw KTsKICAgICAgc3RfYWRkX2RpcmVjdCh0YmwsIHJiX3N0cl9uZXc0KGtleSksIHZhbCk7CiAgICB9 CiAgICBicmVhazsKICBjYXNlIFRfQVJSQVk6CiAgICBhcnkgPSBrZXk7CiAgICBsZW5ndGggPSBS QVJSQVkoYXJ5KS0+bGVuOwogICAgZm9yKG9mcz0wOyBvZnMgPCBsZW5ndGg7IG9mcysrKSB7CiAg ICAgIGtleSA9IFJBUlJBWShhcnkpLT5wdHJbb2ZzXTsKICAgICAgaWYgKFRZUEUoa2V5KSA9PSBU X1NUUklORykgewogICAgICAgIGlmIChzdF9sb29rdXAodGJsLCBrZXksICZ2YWwpKSB7CiAgICAg ICAgICB2YWwgPSByYl9mdW5jYWxsKHZhbCwgaWRfc3VjYywgMCk7CiAgICAgICAgICBzdF9pbnNl cnQodGJsLCBrZXksIHZhbCk7CiAgICAgICAgfSBlbHNlIHsKICAgICAgICAgIHZhbCA9IHJiX2Z1 bmNhbGwoaGFzaCwgaWRfZGVmYXVsdCwgMCk7CiAgICAgICAgICB2YWwgPSByYl9mdW5jYWxsKHZh bCwgaWRfc3VjYywgMCk7CiAgICAgICAgICBzdF9hZGRfZGlyZWN0KHRibCwgcmJfc3RyX25ldzQo a2V5KSwgdmFsKTsKICAgICAgICB9CiAgICAgIH0gZWxzZSB7CiAgICAgICAgaWYgKHN0X2xvb2t1 cCh0YmwsIGtleSwgJnZhbCkpIHsKICAgICAgICAgIHZhbCA9IHJiX2Z1bmNhbGwodmFsLCBpZF9z dWNjLCAwKTsKICAgICAgICAgIHN0X2luc2VydCh0YmwsIGtleSwgdmFsKTsKICAgICAgICB9IGVs c2UgewogICAgICAgICAgdmFsID0gcmJfZnVuY2FsbChoYXNoLCBpZF9kZWZhdWx0LCAwKTsKICAg ICAgICAgIHZhbCA9IHJiX2Z1bmNhbGwodmFsLCBpZF9zdWNjLCAwKTsKICAgICAgICAgIHN0X2Fk ZF9kaXJlY3QodGJsLCBrZXksIHZhbCk7CiAgICAgICAgfQogICAgICB9CiAgICB9CiAgICBicmVh azsKICBkZWZhdWx0OgogICAgaWYgKHN0X2xvb2t1cCh0YmwsIGtleSwgJnZhbCkpIHsKICAgICAg dmFsID0gcmJfZnVuY2FsbCh2YWwsIGlkX3N1Y2MsIDApOwogICAgICBzdF9pbnNlcnQodGJsLCBr ZXksIHZhbCk7CiAgICB9IGVsc2UgewogICAgICB2YWwgPSByYl9mdW5jYWxsKGhhc2gsIGlkX2Rl ZmF1bHQsIDApOwogICAgICB2YWwgPSByYl9mdW5jYWxsKHZhbCwgaWRfc3VjYywgMCk7CiAgICAg IHN0X2FkZF9kaXJlY3QodGJsLCBrZXksIHZhbCk7CiAgICB9CiAgfQogIHJldHVybiBoYXNoOwp9 Cgp2b2lkCkluaXRfaGFzaHN1Y2NnKCkKewogIGlkX2RlZmF1bHQgPSByYl9pbnRlcm4oImRlZmF1 bHQiKTsKICBpZF9zdWNjICAgID0gcmJfaW50ZXJuKCJzdWNjIik7CiAgcmJfZGVmaW5lX21ldGhv ZChyYl9jSGFzaCwic3VjYyEiLCBoYXNoX3N1Y2MsIDEpOwp9Cg --------_3E56479AED4F017A0008_MULTIPART_MIXED_--