building with clang about hyperdex HOT 3 CLOSED

one more issue -
daemon/key_state.cc

barfs tons of errors during compile - important ones are

hyperdex::key_state::client_response
method is not marked const
std::push_heap invalid operands to binary expression

change line 100 , add const to end.

// intentionally backwards so the max heap becomes min heap.
    bool operator < (const client_response& rhs) const

haven't yet tested the binaries :) but something to consider when building with clang

from hyperdex.

update,

This problem is happening because "partitions" is already defined in
/usr/include/c++/v1/algorithm:

template <class ForwardIterator, class Predicate>
    ForwardIterator
    partition(ForwardIterator first, ForwardIterator last, Predicate pred);

The compiler is apparently getting confused, or maybe partition should be explicitly referenced anyway?

This fixes the problem:
admin/hyperspace_builder.cc

569: for (size_t i = 0; i < sp.subspaces.size(); ++i)
570: {
571:    hyperdex::partition(sp.subspaces[i].attrs.size(), in->partitions, &sp.subspaces[i].regions);
572: }

Should compile on gcc, etc, would have to test.

from hyperdex.

HyperDex compiles on clang 3.7 on FreeBSD 9.x. I didn't test 3.4, but I'm open to any patches to make it work on both.

from hyperdex.