Skip to content

Commit

Permalink
refactor: remove outsourced todo markers
Browse files Browse the repository at this point in the history
  • Loading branch information
EagleoutIce committed Sep 23, 2024
1 parent 7618f23 commit bbe25bd
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions src/dataflow/environments/resolve-by-name.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ import type { Identifier, IdentifierDefinition } from './identifier';
import { happensInEveryBranch } from '../info';


// TODO: cache this! => promote environments to classes
/**
* Resolves a given identifier name to a list of its possible definition location using R scoping and resolving rules.
*
Expand All @@ -20,7 +19,6 @@ export function resolveByName(name: Identifier, environment: REnvironmentInforma
do{
const definition = current.memory.get(name);
if(definition !== undefined) {
/* TODO: guard for other control dependencies which are set? */
if(definition.every(d => happensInEveryBranch(d.controlDependencies))) {
return definition;
} else {
Expand Down

0 comments on commit bbe25bd

Please sign in to comment.