Skip to content

Refactor repeated format_shape() helper in linalg into a mix-in #893

@tigercosmos

Description

@tigercosmos

Follow-up from review of #815.

A format_shape() helper -- formatting a SimpleArray shape as "(d0, d1, ...)" for error messages -- is duplicated across several linalg classes (e.g. LuFactorization in cpp/modmesh/linalg/lu_factorization.hpp, and other LA code).

Scope: extract the shared format_shape() logic into a reusable mix-in (or comparable shared base) so the LA classes no longer each carry their own copy.

Raised by @yungyuc in #815.

Metadata

Metadata

Assignees

No one assigned

    Labels

    arrayMulti-dimensional array implementationrefactorChange code without changing tests

    Type

    No fields configured for Bug.

    Projects

    Status
    Todo

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions