diff options
| author | Peter Stephenson <pws@zsh.org> | 2015-10-29 17:06:27 +0000 |
|---|---|---|
| committer | Peter Stephenson <pws@zsh.org> | 2015-10-29 17:06:27 +0000 |
| commit | de9effbce601db7ad7f7f0d0969b70b920b4e371 (patch) | |
| tree | dcd69faaedb1322e922df67ed1285981b4c06d8d /Src/math.c | |
| parent | 300a58f5935631cad660ed238315e68bb86fca8c (diff) | |
| download | zsh-de9effbce601db7ad7f7f0d0969b70b920b4e371.tar.gz zsh-de9effbce601db7ad7f7f0d0969b70b920b4e371.tar.xz zsh-de9effbce601db7ad7f7f0d0969b70b920b4e371.zip | |
37018: Make WARNCREATEGLOBAL more consistent.
Wd don't need separate math handling any more, and can make it output the function name in all cases.
Diffstat (limited to 'Src/math.c')
| -rw-r--r-- | Src/math.c | 26 |
1 files changed, 0 insertions, 26 deletions
diff --git a/Src/math.c b/Src/math.c index eee21e146..37981cf22 100644 --- a/Src/math.c +++ b/Src/math.c @@ -893,25 +893,6 @@ getcvar(char *s) return mn; } - -/* If script execution is inside a function call that hasn't returned, - * return the name of that function. Else return NULL. - */ - -/**/ -static const char * -in_function_call(void) -{ - Funcstack i; - for (i = funcstack; i; i = i->prev) - if (i->tp == FS_FUNC) { - DPUTS(!i->name, "funcstack entry with no name"); - return i->name; - } - - return NULL; -} - /**/ static mnumber setmathvar(struct mathvalue *mvp, mnumber v) @@ -947,13 +928,6 @@ setmathvar(struct mathvalue *mvp, mnumber v) if (noeval) return v; untokenize(mvp->lval); - if (isset(WARNCREATEGLOBAL)) { - const char *function_name; - if (!paramtab->getnode(paramtab, mvp->lval) && - (function_name = in_function_call())) - zwarn("math parameter %s created globally in function %s", - mvp->lval, function_name); - } pm = setnparam(mvp->lval, v); if (pm) { /* |