• last updated 3 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
passing "-debug" flag explicitly to nsf::proc

Added support for trailing non-positional arguments

The change makes the old hack with the own trailing-argument-parser

"ad_arg_parser" obsolete by using directly the c-level argument parser

which is anyhow capable of handling trailing arguments. A few changes

were necessary for api doc to handle there trailing arguments as well.

The proc "db_string" is already converted to use native arguments.

  1. … 7 more files in changeset.
added a comment about the usefulness of ad_arg_parser

reduce the size of the public interface

  1. … 2 more files in changeset.
include package key in proc info

mark functions called only internally as private

  1. … 15 more files in changeset.
just avoid tilde substitution, do not perform sanitizing

sanitizing can lead to file name clashes in case the names

without tilde exists, and they are OS specific. The intention

of "ad_file" is just to be a tilde-proof "file" replacement

  1. … 1 more file in changeset.
Sanitize all trailing tildes from the filename, mapping them to the current folder

Fixes acs-bootstrap-installer.ad_file automated test

improve spelling

  1. … 4 more files in changeset.
allow value checkers in ad_procs

Prefer 'namespace which' over 'info commands', as it is faster (on local tests, around 2x) and returns a single value. Many thanks to Nathan Coulter.

  1. … 58 more files in changeset.
Treat callback hooks and implementations as private api

Reduce acs-tcl.documentation__check_deprecated_see test warnings

  1. … 2 more files in changeset.
introduced "ad_file" as a means to avoid unexpected tilde substitution in file names

  1. … 50 more files in changeset.
deprecate unused function

Restrict switches to not negative Tcl numbers, and avoid non-printable characters as well

Fix typo

Remove private proc 'number_p', used only in 'ad_proc_valid_switch_p', and simplify logic of the latter

Fixed bug while populating the 'api_proc_doc_scripts' nsv on ad_proc, where some procs where not added due to a race condition between the 'acs-automated-testing' and the 'acs-bootstrap-installer' packages. See comments in commit for more details.

improve spelling

  1. … 3 more files in changeset.
Deprecate trivial empty_string_p

improve spelling

improve spelling

  1. … 15 more files in changeset.
add missing proc_doc

Document the use of the parameter 'args' containing a list of a variable number of arguments

Whitespace changes

Fix misleading and uncoherent 'ad_dispatch' proc documentation, discovered using the 'documentation__check_parameters' test.

procdoc change: Preserve pre-existing values, e.g. from testcases, which might be loaded in a different order (testcase might be loaded before proc)

.CVS: ----------------------------------------------------------------------

remove dead code

improve spelling

  1. … 2 more files in changeset.