[kixelated]

Problem

std::mem::needs_drop::<T>() is refused as an unsupported call, while size_of::<T>() translates as its value. Found while reproducing moq-dev/rust2#480.

Approach

Plan it as an m3 quest, quest/m3/ts-needs-drop.md: translate it as rustc's constant for each instance, proven by a differential test against native Rust.

Impact

• None: quest tree only.

Alternatives

• Fold it into fix(ts): fold a comparison of two constants a generic function's instance makes #480. Rejected: it is a separate refusal, not a comparison of constants.

Follow-ups

• None.

Review

mechanical fix by quest land watch: merged main at c8e039785f3a40be3bd8b0c93c7d5e9f9eb63379 with the quest merge driver (head 5e7d346)

Skipped: a quest-only change, too trivial to review. just check passed.

🤖 Generated with Claude Code

[coderabbitai]

Important

• Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Organization UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: 9fb3bf7f-69db-4bc6-83f1-bff2235b5f54

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

• X
• Mastodon
• Reddit
• LinkedIn

CodeRabbit Verify

• Run CodeRabbit Verify to test the changes in this pull request.

[moq-bot]

quest land watch: the merge queue removed this pull request at 6a09a29 because it conflicts with main (merge_conflict).

The land fixer works on it in the background, and the watcher re-enqueues it at the back of the queue once it is fixed or judged flaky, or hands it back here with what it needs.

(written by quest land watch)

[moq-bot]

quest land watch: armed auto-merge on this pull request at 5e7d346, so it joins the back of the merge queue once its required checks pass, with main at c8e0397 merged in by the quest merge driver.

(written by quest land watch)