Results 215 comments of Jochen Topf

I am testing the [Clara](https://github.com/catchorg/Clara) in vtzero. But for protozero we need a single `-` to work. See https://github.com/catchorg/Clara/issues/57.

No. This is more theoretical.

@flohoff I recommend using backports for stretch which will give you current protozero and libosmium. Searching `../protozero` is an ugly hack, not something the CMake config should do anyway. I...

The best I can come up with is: ``` c++ // True if Predicate matches for none of the types T from Ts template struct static_none_of_binary : std::is_same {}; template...

Currently there is still the public `get_type_index()` function, so this internal index is visible outside. I suggest we mark this function as deprecated and remove it in 2.0 and then...

I fixed this for protozero in https://github.com/mapbox/protozero/commit/d6e2b35fc2794de937a85aa1755051a30176fc06.

Currently `class variant` has data members `std::size_t type_index` and `data_type data`. Here `sizeof(data_type)` will be the size of the largest type the variant includes. But together they will be at...

I we are only thinking about the size of the index, I think we should just go with `int8_t`. I can't think of a use case where more than 255...

This should be an easy change once we have a more complete set of tests.

I don't feel comfortable having such complex code (especially in areas with compiler problems like #86) without any tests. And it is really easy to get the implementation of the...