• Dridi Boukelmoune's avatar
    Defer the illegal write check a bit · ed36b638
    Dridi Boukelmoune authored
    After the initial discussion from #3163, and looking more closely at how
    variable access is handled in subroutines I noticed a discrepancy.
    
    Setting a read only variable like obj.ttl in vcl_recv would result in
    a misleading error message explaining that it is read only, instead of
    simply not available.
    
    This change defers the illegal write check, registering unconditionally
    that the symbol was used in a set action. As a result we always get the
    correct error message but depending on whether this is happening in a
    vcl_ or custom subroutine we may either get "in subroutine" or "from
    subroutine" in the error message. A minor discrepancy probably worth
    getting rid of the prior inconsistency.
    
    This is covered by the v21 test case.
    ed36b638
Name
Last commit
Last update
.circleci Loading commit data...
.github Loading commit data...
bin Loading commit data...
doc Loading commit data...
etc Loading commit data...
include Loading commit data...
lib Loading commit data...
m4 Loading commit data...
man Loading commit data...
tools Loading commit data...
.dir-locals.el Loading commit data...
.envrc Loading commit data...
.gitignore Loading commit data...
.lgtm.yml Loading commit data...
.syntastic_c_config Loading commit data...
.travis.yml Loading commit data...
CONTRIBUTING Loading commit data...
ChangeLog Loading commit data...
INSTALL Loading commit data...
LICENSE Loading commit data...
Makefile.am Loading commit data...
README.Packaging Loading commit data...
README.rst Loading commit data...
autogen.des Loading commit data...
autogen.sh Loading commit data...
configure.ac Loading commit data...
flint.lnt Loading commit data...
varnish-legacy.m4 Loading commit data...
varnish.m4 Loading commit data...
varnishapi-uninstalled.pc.in Loading commit data...
varnishapi.pc.in Loading commit data...
vtc.am Loading commit data...