Closed sesevgen closed 5 years ago
This pull request introduces 1 alert when merging ab05f24acb142f6e32c68752f8d6b3d3464a180a into 2dfafe6087743e0de0f169971ab89d218f9a9cec - view on LGTM.com
new alerts:
Comment posted by LGTM.com
This pull request introduces 4 alerts when merging ac28d32454d0e1f94b6b2f04464ed3ee35190088 into 2dfafe6087743e0de0f169971ab89d218f9a9cec - view on LGTM.com
new alerts:
Comment posted by LGTM.com
I took a stab at this after our discussion @tkornut . Let me know what you think.
I hope to add a command line dictionary parser later, so any default parameter can be overridden by the user in a smart way.