Skip to content

fix(x64-simulator): use actual_index for outputs_data in SOURCE_GROUP_OUTPUT#46

Merged
quake merged 2 commits intonervosnetwork:mainfrom
quake:main
Jan 27, 2026
Merged

fix(x64-simulator): use actual_index for outputs_data in SOURCE_GROUP_OUTPUT#46
quake merged 2 commits intonervosnetwork:mainfrom
quake:main

Conversation

@quake
Copy link
Member

@quake quake commented Jan 27, 2026

Fix incorrect cell data fetching when using Source::GroupOutput. The code
was using the group-relative index instead of the transaction-relative
actual_index to retrieve outputs_data, causing wrong cell data to be
returned when the type script only appears on a subset of outputs.

quake added 2 commits January 27, 2026 18:40
…_OUTPUT

Fix incorrect cell data fetching when using Source::GroupOutput. The code
was using the group-relative index instead of the transaction-relative
actual_index to retrieve outputs_data, causing wrong cell data to be
returned when the type script only appears on a subset of outputs.
@quake quake requested review from XuJiandong, Copilot and mohanson and removed request for Copilot January 27, 2026 09:49
Copy link
Collaborator

@mohanson mohanson left a comment

Choose a reason for hiding this comment

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

LGTM

@quake quake merged commit 6f635b4 into nervosnetwork:main Jan 27, 2026
7 checks passed
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.

2 participants