Skip to content

xexpression_holder generic constructor #2969

Closed

Description

@JohanMabille

xepxression_holder provides a generic non-constrained constructor. The problem is that this constructor is preferred over the copy constructor when copying non const xexpression_holder objects, leading to a segfault.

This constructor should be constrained to not accept xexpression_holder objects.

Activity

  1. GinkelKostia commented on Nov 11, 2026

    @GinkelKostia

    Hi!

    I would like to contribute to this issue.

    I am a C++ developer with experience in modern C++, templates, CMake and unit testing. I would like to investigate the constructor overload resolution issue and work on a fix with appropriate regression tests.

    Is this issue available for contribution? I would be happy to submit a pull request.

    Thank you!

  2. JohanMabille commented on Nov 12, 2026

    @JohanMabille
    MemberAuthor

    Hi,

    Actually this issue was fixed in #2970 and I forgot to close it. But they are a lot of other issues opened if you want to contribute. Some of them have been opened for a while and may have been fixed with the different refactorings, but it might be worth giving a try to reproduce them and investigate if they're still here.

    Thanks!

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

Metadata

Metadata

Assignees

No one assigned

    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