Bug or outdated/wrong comment?

[cloudhan]

When I skimming through some code, I find

kaldi/src/util/stl-utils.h

Lines 98 to 109 in 8ce3a95

	template<class T>
	void CopySetToVector(const unordered_set<T> &s, std::vector<T> *v) {
	// adds members of s to v, in sorted order from lowest to highest
	// (because the set was in sorted order).
	KALDI_ASSERT(v != NULL);
	v->resize(s.size());
	typename unordered_set<T>::const_iterator siter = s.begin(), send = s.end();
	typename std::vector<T>::iterator viter = v->begin();
	for (; siter != send; ++siter, ++viter) {
	*viter = *siter;
	}
	}

It states (because the set was in sorted order). But the set is an unordered_set, is it a bug or simply forgetting to remove the comment when copying code from the set version of CopySetToVector?

[danpovey]

yes the comment is wrong. if you make a PR to remove it I would merge, but won't leave the issue up (too minor)