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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.