Skip to content

refactor: Convert nullable Embind types to use std::optional #51

@coderabbitai

Description

@coderabbitai

Background

Currently, we register nullable types in Embind using custom type definitions that map to 'T | null' in TypeScript. For example:

  • FilteredLogEventMapTsType -> number[] | null
  • LogEventIdxTsType -> number | null

Embind now provides built-in support for std::optional<T> through the register_optional() function, as documented here.

Proposed Changes

Convert all nullable type registrations to use std::optional<T> with register_optional().

References

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions