id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc	revision
353	Clarify typenames in NetworkSimplex	Peter Kovacs	Peter Kovacs	"In [f3bc4e9b5f3a], the following typedefs
{{{
#!cpp
  typedef std::vector<char> CharVector;
}}}
was replaced with
{{{
#!cpp
  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."	defect	closed	major	LEMON 1.2 release	core	hg main	fixed