Custom Query (545 matches)

Current clang version complains about the following function definition in concepts::Readmap.

      Value operator[](const Key &) const {
        return *(static_cast<Value *>(0)+1);
      }

The attached patch fixes it by changing to

        return Value();

But this means that ReadMap::Value must be default constructible. (Thus tests/map_tests.cc had to be changed accordingly.) Unfortunately I couldn't come up with any better solution.

Any idea?

In [f3bc4e9b5f3a], the following typedefs

  typedef std::vector<char> CharVector;

was replaced with

  typedef std::vector<char> BoolVector;
  // Note: vector<char> is used instead of vector<bool> for efficiency reasons

in all min cost flow classes to indicate that these types are used to store boolean values and char is only used for efficiency reasons. But in NetworkSimplex, it is not true. There are two vectors: the first one stores boolean values, but the other one stores {-1, 0, 1} values. Using the name BoolVector for the latter case is misleading. So I was not careful enough when I made those renamings, but the attached patch separates these two use cases.

This task should be done after the 1.0 release branch will have been created.

Documentation contains references to tools that has not yet been ported from the svn. (For example there are empty modules is the doc.) These parts must be removed before the release.