Add documentation comment to op.h to clarify that LOGOP's ->op_other is in execution order pointer, not a tree-structural one

1 files changed, 6 insertions, 0 deletions

diff --git a/op.h b/op.h
index 6d8729680b..c6de0d19fa 100644
--- a/op.h
+++ b/op.h
@@ -224,6 +224,12 @@ struct binop {
 struct logop {
     BASEOP
     OP *	op_first;
+
+    /* Note that op->op_other is the *next* op in execution order of the
+     * alternate branch, not the root of the subtree. I.e. imagine it being
+     * called ->op_otherfirst.
+     * To find the structural subtree root (what could be called
+     * ->op_otherroot), use OpSIBLING of ->op_first  */
     OP *	op_other;
 };