search

suewonjp / tclsh-wrapper

A tiny wrapper for Tcl/Tk's tclsh and wish
Apache License 2.0
22 stars 4 forks source link

Code organization #9

Closed jsuntheimer72 closed 6 years ago

jsuntheimer72 commented 6 years ago

Hi!

So currently all tclsh-wrapper commands are in bag-of-procs namespace. There are only so many, the complexity is not insurmountable or anything, but to dip your toe into tclsh-wrapper is swim in it's pool. There are two obvious 8.6 solutions: child namespaces and tcloo.
Although I want to justify this at length, I won't, instead when next you hear from me, I am going to present a tcloo-ified tclsh-wrapper, and we all can discuss it as a proposal. If you prefer to use namespaces, I don't mind at all if this effort is ultimately in vain :)

suewonjp commented 6 years ago

Thanks.

I don't prefer namespaces. Feel free to apply tcloo to the existing code.

I'm not familiar with tcloo, though, I don't mind learning it.

jsuntheimer72 commented 6 years ago

closing this