nsf

Clone Tools
  • last updated 10 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
remove shadowing variable

reduce implicit conversions

transitional fix for tcl87a2 problem

substdefault code cleanup

extend regression test, make assumptions explicit

- remove stripping of substdefault from properties

- add handling of extended substdefault options to per-object variables - improve default value checking for slot-less variables - transfor associated array into a dict - extend regression tests

delete pre-existing commands explicitly

- provide fully qualified names to Tcl_ProcObjCmd() to avoid potential problems in newer Tcl versions

Preserve changes to monitor the interaction between MakeProc and e76a04e48fb1b850

- make it possible to use error code for cmd result comparison - add some NSF specifc error code

provide error hint for "invalid value constraints" errors

add error message, when present

Reduce number of strlen() operations

whitespace changes

Fold "substdefault" and "substdefaultoptions" into a single parameter option "substdefault"

* nsf.c (ParamOptionParse): Fixed typo in error msg.

* nsf.c (NSCheckNamespace): Simplify resource management of a DString.

* nsf.c (ParamSetFromAny2): Fix inline comment.

* nsf.c (ParamOptionParse, ParamDefinitionParse, ParamDefsParse), parameters.test: Small "type=" converterArg reform. Provided additional parameter in param-parser machinery, to communicate a (namespace) qualifier into ParamOptionParse. This way, the unqualified type=* values are now expanded to qualified names (definition scope): method parameters, nsf::is, nsf::parseargs, setter methods. Type=* in object parameters are currently expanded at the slot level, could also be added. New tests were added, existing ones adjusted to reflect the new behaviour.

Adding test cases

Adjust tests to match new behavior

Fix collateral in ParseParamOptions

Fix subtle bug

Done with test cases; some cleaning up

- new command "tnsf::definitionnamespace" - experiment with command in regression test

fix typo in comment

- implement substdefaultoptions (for now, just providing the bitmask) - add substdefault.test - rename static function ParamParse() to ParamDefinitionParse() - break overlong lines

    • -0
    • +208
    /tests/substdefault.test
Add some general to-dos on the way

Added missing tests, and cases; WIP (two to-dos, and cleanup pending)