From a85d92b2eedba2340caa84e70fef48d03aae2347 Mon Sep 17 00:00:00 2001 From: Vern Paxson Date: Fri, 7 Jan 2022 11:49:03 -0800 Subject: [PATCH] minor commenting clarifications --- doc | 2 +- src/Frame.h | 5 +++-- src/Var.cc | 4 ++++ 3 files changed, 8 insertions(+), 3 deletions(-) diff --git a/doc b/doc index 1263619ce3..0cebe39463 160000 --- a/doc +++ b/doc @@ -1 +1 @@ -Subproject commit 1263619ce3279415314355032b855206f7e3e632 +Subproject commit 0cebe394636728788b2f880d7d8ec661bf22824e diff --git a/src/Frame.h b/src/Frame.h index 395cb8009e..d67929fb2b 100644 --- a/src/Frame.h +++ b/src/Frame.h @@ -54,8 +54,9 @@ public: Frame(int size, const ScriptFunc* func, const zeek::Args* fn_args); /** - * Deletes the frame. Unrefs its trigger, the values that it - * contains and its closure if applicable. + * Deletes the frame. Unrefs its trigger (implicitly, since it's an + * IntrusivePtr), and the values that the frame contains and its + * closure if applicable. */ virtual ~Frame() override; diff --git a/src/Var.cc b/src/Var.cc index 5437b11c33..2cf367228e 100644 --- a/src/Var.cc +++ b/src/Var.cc @@ -722,6 +722,10 @@ TraversalCode OuterIDBindingFinder::PostExpr(const Expr* expr) return TC_CONTINUE; } +// The following is only used for debugging AST duplication. If activated, +// each AST is replaced with its duplicate. In the absence of a duplication +// error, this shouldn't change any semantics, so running the test suite +// with this variable set can find flaws in the duplication machinery. static bool duplicate_ASTs = getenv("ZEEK_DUPLICATE_ASTS"); void end_func(StmtPtr body, bool free_of_conditionals)