Skip to content

Discussion: Unreachable if(TRUE) else block in cansim_metadata.R #151

@dshkol

Description

@dshkol

Summary

There's an if (TRUE) condition with an else branch that can never execute.

Current State

  • File: R/cansim_metadata.R:37-56 (approximate lines)
  • Code has if (TRUE) { ... } else { ... }
  • The else branch is dead code

Questions

  • Was this intentional debugging code that should be removed?
  • Is the else branch an incomplete feature?
  • What was the else branch meant to do?

Options

  1. Remove the condition - Just keep the if-block contents
  2. Complete the else branch - If there's intended functionality
  3. Parameterize - Turn into a real condition if needed

From code audit - needs context on original intent

Metadata

Metadata

Assignees

No one assigned

    Labels

    questionFurther information is requested

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions