Skip to content

Fix cpp issues for no-handler-scopery work - #487

Open
tjpalmer wants to merge 3 commits into
mainfrom
no-handler-scopery-cpp
Open

tjpalmer wants to merge 3 commits into
mainfrom
no-handler-scopery-cpp

Conversation

@tjpalmer

@tjpalmer tjpalmer commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

val paramTypes = ctxSig.requiredInputTypes.drop(
if (sig.hasThisFormal || isSuperCall) 1 else 0,
)
val paramTypes = ctxSig.requiredInputTypes

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Apparently, contextualized signatures drop this automatically now.

throw TemperBubble();
}
return promise->get();
}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

New way of doing this for exceptioning backends.

connectedNames = listOf("UnpackOkResult"),
builtinOperatorId = BuiltinOperatorId.UnpackOkResult,
useDot = true,
)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

These all map in pretty straightforward ways. We use them as values instead of pointers, though, so I added that useDot option.

Ft.TypesIntBasics,
Ft.TypesIntLimits,
Ft.TypesListEmpty,
)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I think this matches what was there before #474.

@tjpalmer
tjpalmer requested a review from ShawSumma September 12, 2026 02:46
@tjpalmer
tjpalmer marked this pull request as ready for review September 12, 2026 02:46
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.

1 participant