Skip to content

refactor(clp-s): Use the create() pattern in clp_s::TimestampDIctionaryWriter and clp_s::ArchiveWriter. #2007

@gibber9809

Description

@gibber9809

Request

Currently clp_s::TimestampDictionaryWriter forwards errors from ystdlib::error_handling::Result by throwing, which is a bit awkward. It would be better to switch to a factory method that forwards the errors, as requested in the review for #1788.

Possible implementation

Implement a create() method that forwards a ystdlib::error_handling::Result<clp_s::TimestampDictionaryWriter> and ystdlib::error_handling::Result<clp_s::ArchiveWriter> respectively.

This is currently blocked by some missing move constructors as described in #2006.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions