SUMMARY OF CODE REVIEW -- 08/21/1996: Top-level Functionality of the Execution Engine - exec.h/c, ops.c
Presenter: Ramana Yerneni
Note-taker: Joachim Hammer
Items of discussion:
In exec.h/c
- Inclusion of btable.h not necessary since the graph traversal
functions have been moved. The same is true for class bTable.
- Rename headtemp and tailtemp to avoid confusion
with wrapper templates. Suggested replacement names are qhead and
qtail.
- sourceanswer can only be deleted along with the answer.
- Investigate the difference bwteen NULL bindings and empty table.
- In case of an error, delete bindings before the function break.
In ops.c
- Need function for cleaning up result tables (for every node in the
graph)
- colindex is superfluous; just cehck for -1
Joachim Hammer