Skip to content

Drop the unused Nullable parameter of the conversion accessor - #92

Open
mingxwa wants to merge 1 commit into
ngcpp:feature/v5from
mingxwa:user/mingxwa/drop-cast-nullable
Open

mingxwa wants to merge 1 commit into
ngcpp:feature/v5from
mingxwa:user/mingxwa/drop-cast-nullable

Conversation

@mingxwa

@mingxwa mingxwa commented Sep 26, 2026

Copy link
Copy Markdown
Member

Refactoring. No functional change.

cast_dispatch_base took a Nullable parameter so that the accessor of
substitution_dispatch could convert an empty proxy to an empty proxy
without invoking the dispatch. substitution_dispatch has been gone
since ngcpp#84, and both remaining users, implicit_conversion_dispatch and
explicit_conversion_dispatch, pass false, so the branch it guarded was
never instantiated.

This branch has not been deployed

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant