Paul Fultz II

Only inherit when the class is empty

1

The main reason for inheritance is to take advantage of EBO. So it might be best to only inherit when classes are empty. This is related to issue #129, and...

The decorator created by the decorate adaptor should allow for constraints and multiple parameters

4

One way to do this is to have a second function passed to `decorate`, which will construct or pack the data. It can also be used to constrain the decorator.

Consider renaming apply to unpack

4

Generally, `apply` is synonymous with `invoke`. MPL library uses `apply` to invoke a metafunction or lambda. Boost.Hana's `apply` works just like `std::invoke`. However, your `apply` unpacks the elements of a...

Doing `match(tok, "**;")` will match until the semicolon. This also fixes the incorrect slicing for `!!` match as well.

This is defined for every test, so it should be moved to usage requirements for data_time.

This moves the condition into the if statement so that short-circuiting can skip it when the ID is negative. Otherwise it can use a negative index for `LocalSLocEntryTable[LastFileIDLookup.ID]`.