Re: PATCH: fix isbuild, ishost, istarget defaults
Jacob Bachmeyer <[email protected]>
| Newsgroups | gmane.comp.sysutils.dejagnu.general |
|---|---|
| Message-ID | <[email protected]> |
Ben Elliston wrote:
> The documentation implies that you can call procs isbuild, ishost and
> istarget with no argument to return these triplets. However, passing
> "" is not the same as passing nothing in Tcl:
>
> % proc foo {arg} { puts $arg }
> % foo ""
>
> % foo
> wrong # args: should be "foo arg"
>
> This patch fixes these procs so that they can be called without an
> argument. This remains compatible with the existing behaviour.
>
> OK to commit?
>
I have two comments, both about the documentation changes:
First, the documentation references to "NULL" or "NULL string" make no
sense in Tcl and *should* be changed (as this patch starts to do) to
"empty string". (In other words, are there more places to change?
There is no "NULL" in Tcl.)
Second, instead of "If called with no arguments", the documentation
should say "If called with no arguments or an empty string", since that
is what the function actually examines.
-- Jacob