Skip to content

Remove stale commented-out input-size-limit code in AssemblyCompression.cs#11677

Open
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-finder-remove-stale-code
Open

Remove stale commented-out input-size-limit code in AssemblyCompression.cs#11677
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-finder-remove-stale-code

Conversation

Copilot AI commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

AssemblyCompression.cs carried a // TODO and a block of dead, commented-out code for an "input assembly size limit" feature that was never enabled, including an obsolete "(May 2020)" sizing note. The commented-out InputAssemblySizeLimit constant and its sole usage were both inert clutter.

Changes

  • Removed the stale // TODO, the explanatory comment, and the commented-out InputAssemblySizeLimit constant.
  • Removed the commented-out InputAssemblySizeLimit usage in Compress.
  • Left CompressionResult.InputTooBig and its live switch case untouched, keeping this purely comment-only with zero behavior change.

This is a comment-only deletion — no executable code, APIs, or language features changed.

…imit code

Co-authored-by: jonathanpeppers <840039+jonathanpeppers@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove stale commented-out input assembly size limit code Remove stale commented-out input-size-limit code in AssemblyCompression.cs Jun 17, 2026
Copilot AI requested a review from jonathanpeppers June 17, 2026 03:00
@jonathanpeppers jonathanpeppers marked this pull request as ready for review June 17, 2026 03:01
Copilot AI review requested due to automatic review settings June 17, 2026 03:01

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes stale commented-out code and outdated notes from AssemblyCompression.cs, keeping the AssemblyCompression utility focused on the active compression implementation without altering behavior.

Changes:

  • Deleted the old // TODO and explanatory comment block about an unused “input assembly size limit”.
  • Removed the commented-out InputAssemblySizeLimit constant and its commented-out usage in Compress.
  • Left the existing CompressionResult.InputTooBig enum value and switch handling unchanged (no functional change).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants