Changes by: Jason J. Hickey (jyh at cs.caltech.edu)
Date: 2007-08-06 14:14:04 -0700 (Mon, 06 Aug 2007)
Revision: 11850
Log message:
The hoisting Aleksey added in revs 11448-11849 was not right--
the following expression doesn't make sense:
venv_export_venv venv_orig venv_src
This is because the two environments have different this.
venv_orig has the caller's "this", venv_src has the callee's "this".
The two objects are different, so the export makes no sense.
NOTE: disabled the path check, going back to the original policy
that hoisting overrides export. This is matter of policy, so I
think it is better to keep the code simple, and discuss it.
Also, changed the tests a little, so an exit code of 2
is a "note", not an "error".
Changes | Path(relative to omake-branches/0.9.8.x) |
+17 -36 | src/env/omake_env.ml |
+1 -1 | src/env/omake_env.mli |
+1 -1 | src/eval/omake_eval.ml |
+19 -3 | test/OMakefile |
+4 -1 | test/object/Test08 |
+3 -2 | test/object/Test10 |
+4 -1 | test/object/Test14 |
+4 -1 | test/object/Test15 |
+1 -1 | test/simple/Test8/run.osh |
+2 -2 | test/targets/Test3/run.osh |