Filip Sajdak comments

Results 129 comments of


                                            Filip Sajdak

[SUGGESTION] Add possibility of using lambda in is-statements (inspect and if statements)

@JohelEGP `labda_wrapper` requires ```cpp requires { &F::operator(); } ``` It means that the below code will work in the `is` statement: ```cpp constexpr auto in = [](auto min, auto max)...

[SUGGESTION] Add possibility of using lambda in is-statements (inspect and if statements)

I will test that direction. Unfortunately, `std::tuple` cannot be used. As I described in the PR description: > lambda_wrapper shall use std::tuple to store arguments that shall go to capture...

[SUGGESTION] Add possibility of using lambda in is-statements (inspect and if statements)

@hsutter, your change enables using lambda in inspect expressions but not in if statements. My solution worked for both cases ;) ```cpp #include #include constexpr auto less_then = [](int value)...

[SUGGESTION][FIX] Add pointer deduce type support

Replaced by https://github.com/hsutter/cppfront/pull/196

[SUGGESTION][REFACTOR] Rework `is<T>(std::variant)` and `as<T>(std::variant)` to handle variadic number of types

Ensure that call to `std::holds_alternative` is well formed by checking if `T` appears exactly once on the type lists Checks if `count_t == 1` before calling `std::holds_alternative`: ```cpp // std::holds_alternative...

[SUGGESTION][REFACTOR] Rework `is<T>(std::variant)` and `as<T>(std::variant)` to handle variadic number of types

If the variant contains one type multiple times, the `is()` & `as()` methods will end up with static_assert failure.

[SUGGESTION][REFACTOR] Rework `is<T>(std::variant)` and `as<T>(std::variant)` to handle variadic number of types

The currently proposed change will make that code illegal. If you want to support that I can rework that to iterate over types on the list - that will be...

[SUGGESTION][REFACTOR] Rework `is<T>(std::variant)` and `as<T>(std::variant)` to handle variadic number of types

Just a side note. The code you have provided does not work due to the current implementation of `CPP2_UFCS` that fails with an error. I've run into this error before:...

[SUGGESTION][REFACTOR] Rework `is<T>(std::variant)` and `as<T>(std::variant)` to handle variadic number of types

@hsutter If you want to keep the current behavior please take a look at https://godbolt.org/z/vGnzEWeYv The only difference is that it is not returning `nonsuch` instead of that it throws...

[SUGGESTION][REFACTOR] Rework `is<T>(std::variant)` and `as<T>(std::variant)` to handle variadic number of types

@hsutter, regarding the problem with the `CPP2_UFCS` macros, we can introduce another macro for templated methods/functions that we can use when cppfront identifies that the function/method has template arguments. Prototype...