Skip to content

Conversation

@fbusato
Copy link
Contributor

@fbusato fbusato commented Nov 19, 2025

Description

The CCCL host API should use the exception mechanism to report errors.

Changes introduced by this PR:

  • Adds _CCCL_THROW_IF to throw an exception when preconditions are not meet.
  • Applies _CCCL_THROW_IF to make_tma_descriptor to replace _CCCL_VERIFY.

@fbusato fbusato self-assigned this Nov 19, 2025
@fbusato fbusato requested a review from a team as a code owner November 19, 2025 00:08
@fbusato fbusato added the 3.2.0 Targeted for 3.2.0 release label Nov 19, 2025
@fbusato fbusato requested a review from a team as a code owner November 19, 2025 00:08
@fbusato fbusato added this to CCCL Nov 19, 2025
@github-project-automation github-project-automation bot moved this to Todo in CCCL Nov 19, 2025
@cccl-authenticator-app cccl-authenticator-app bot moved this from Todo to In Review in CCCL Nov 19, 2025
@github-actions

This comment has been minimized.

@jrhemstad jrhemstad requested a review from pciolkosz November 19, 2025 02:35
@github-project-automation github-project-automation bot moved this from In Review to In Progress in CCCL Nov 19, 2025
@davebayer
Copy link
Contributor

Also, docs update is missing

@github-actions

This comment has been minimized.

@fbusato fbusato requested a review from davebayer November 21, 2025 17:15
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@fbusato
Copy link
Contributor Author

fbusato commented Nov 21, 2025

Reorganized the headers and a few functions to avoid circular dependencies.

@fbusato fbusato requested a review from davebayer November 21, 2025 23:42
@github-actions

This comment has been minimized.

@davebayer davebayer requested a review from miscco November 24, 2025 10:43
@fbusato fbusato requested review from davebayer and miscco November 24, 2025 19:44
@github-actions
Copy link
Contributor

😬 CI Workflow Results

🟥 Finished in 2h 00m: Pass: 45%/90 | Total: 1d 23h | Max: 1h 50m | Hits: 85%/58400

See results here.

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

Labels

3.2.0 Targeted for 3.2.0 release

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

4 participants