bowenfu / matchit.cpp Goto Github PK

This is a follow up task of #95

matchit.txt

This is similar to ref (mut) patterns in rust.

@gnuherdnix 's original sample
https://godbolt.org/z/jjh6Kf1b8

Adapted:
https://godbolt.org/z/fdrPac3nM

Describe the bug
CMakeLists.txt is only compatible with use via FetchContent, not with the default idiomatic method of installing the package.

Additional context
Read about CMake installation support, for example, here.

Optimize the memory usage and save some copies.

Also pave the path to better support mutation in the handler.

As a conan user I wanted to have this library awailable via conan-center-index. So I made a PR in there.

Could having to use id variables be replaced with lambda parameters?
eg: instead of this

Id<string> text;
match(variant)(
pattern | as<string>(text) = [&] {
            std::cout << "Text message: " << *text << std::endl;
        })

this

match(variant)(
pattern | as<string> = [](const string& text) {
            std::cout << "Text message: " << text << std::endl;
        })

https://godbolt.org/z/41KWjcaEx

Describe the solution you'd like
I would like to see this library added as a package for vcpkg.

Describe alternatives you've considered
Adding a header file is extremely convenient, but adding the package to vcpkg would provide automatic fixes and features for users.

Additional context
Since the library uses CMake already, I think the packaging for vcpkg shouldn't be difficult.

Thanks for the great effort!