Address review comments

rust/ql/lib/codeql/rust/elements/internal/VariableImpl.qll

@@ -381,7 +381,9 @@ module Impl {
   class NestedFunctionAccess extends LocalAccess {
     private Function f;
 
+    NestedFunctionAccess() { f = super.getLocal().getDefiningNode() }
+
     /** Gets the function being accessed. */
-    Function getFunction() { result = super.getLocal().getDefiningNode() }
+    Function getFunction() { result = f }
   }
 }

shared/namebinding/codeql/namebinding/LocalNameBinding.qll

@@ -46,6 +46,10 @@ signature module LocalNameBindingInputSig<LocationSig Location> {
    *   // x is not in scope here
    * }
    * ```
+   *
+   * If a local declaration inside the condition is a shadowing sibling declaration
+   * (see below), then it should use the declaration itself as scope, otherwise it
+   * should use the condition as scope.
    */
   class Conditional extends AstNode {
     /** Gets the condition of this conditional. */
@@ -153,8 +157,6 @@ module LocalNameBinding<LocationSig Location, LocalNameBindingInputSig<Location>
       implicitDeclInScope(_, this)
       or
       isTopScope(this)
-      or
-      lookupStartsAt(_, this)
     }
   }