wduquette / molt

Embeddable TCL Interpreter for Rust applications
BSD 3-Clause "New" or "Revised" License
105 stars 13 forks source link

Performance Improvements and New Features in Forked Version #109

Open zao111222333 opened 4 months ago

zao111222333 commented 4 months ago

Hey,

Thank you for creating this awesome work!

I’ve made some improvements while using this crate and wanted to share my fork. I would be delighted if some of my changes could be merged into this repo.

Some noticeable features include:

Lastly, I evaluated the forked version against molt 0.3.2. Here are the results (note that performance is influenced by real-time CPU workload, and the values represent general cases):

molt-forked 0.4.0 (time unit in Nanos) molt 0.3.2 Speedup (×) Benchmark
89 208 2.34 ok-1.1 ok, no arguments
90 207 2.3 ok-1.2 ok, one argument
97 219 2.26 ok-1.3 ok, two arguments
119 209 1.76 ident-1.1 ident, simple argument
209 402 1.92 incr-1.1 incr a
158 311 1.97 set-1.1 set var value
201 348 1.73 list-1.1 list of six items

Thank you for considering these changes! If could, I will organize the changes and make PR.

wduquette commented 4 months ago

Thank you for your hard work! I am no longer working on Molt, and have not since, I think, 2020. The license is BSD; if you would like to continue development of your own fork, you have my blessing.

zao111222333 commented 4 months ago

Thank you for your hard work! I am no longer working on Molt, and have not since, I think, 2020. The license is BSD; if you would like to continue development of your own fork, you have my blessing.

Thank you again for your support and creating such a solid foundation with Molt! If you ever decide to return or have any suggestions, I’d be delighted to hear from you.