First attempt with macros - [merged]

[slrtbtfs]

Merges macros -> main

First experiments towards #37.

See the tests for how that syntax could be used.

[slrtbtfs]

added 1 commit

• 7e6a721f - simplify syntax

Compare with previous version

[slrtbtfs]

requested review from @bjthehun

[slrtbtfs]

In GitLab by @bjthehun on Dec 15, 2022, 10:22

Commented on src/graph_backends/filter_map.rs line 161

We could add more extensive documentation here (i.e. how to invoke this macro, its variants, internal workings, example/doc-tests...)

[slrtbtfs]

In GitLab by @bjthehun on Dec 15, 2022, 10:24

Commented on src/graph_backends/filter_map.rs line 170

At least on my machine & rustc 1.68; match syntax works fine:

match node {
    $node_pattern => Some(node),
    _ => None,
}

[slrtbtfs]

In GitLab by @bjthehun on Dec 15, 2022, 10:32

Commented on src/graph_backends/filter_map.rs line 160

Maybe we could move macros into their own module, i.e. query.

[slrtbtfs]

fixes #37

[slrtbtfs]

added 1 commit

• 32033a6b - Use match clause

Compare with previous version

[slrtbtfs]

This macro is specific to filter map, so I think we can keep it there for now. That might change if we end up having more macros.

[slrtbtfs]

hm, now it works for me, too.

[slrtbtfs]

In GitLab by @bjthehun on Dec 21, 2022, 10:08

resolved all threads

[slrtbtfs]

In GitLab by @bjthehun on Dec 21, 2022, 10:08

approved this merge request

[slrtbtfs]

In GitLab by @bjthehun on Dec 21, 2022, 10:08

mentioned in commit f617cdbcf6db4139826f579262ca104e55d4ad60