Defining outside loop?

test.cpp

inline std::vector<HE_Edge*> HE_Selection::getOuterEdges() {

	HE_Selection copy = getCopy();
	copy.collectEdges();
	
	const std::vector<HE_Edge*>& copy_edges = copy.getEdgesRef();
	const std::vector<HE_Edge*>::const_iterator it = copy_edges.begin();
	HE_Edge* e = NULL;
	HE_Face* f1 = NULL;
	HE_Face* f2	= NULL; 
	
	std::vector<HE_Edge*> result;	
	while(it != copy_edges.end()) {
		e = (*it);
		f1 = e->getFirstFace();
		f2 = e->getSecondFace();
		if( (f1 == NULL) || (f2 == NULL) || (!containsFace(f1)) || (!constainsFace(f2)) {
			result.push_back(e);
		}
	}
	return result;

}


// ---------- or inside loop

inline std::vector<HE_Edge*> HE_Selection::getOuterEdges() {

	HE_Selection copy = getCopy();
	copy.collectEdges();
	
	const std::vector<HE_Edge*>& copy_edges = copy.getEdgesRef();
	const std::vector<HE_Edge*>::const_iterator it = copy_edges.begin();
	HE_Edge* e = NULL;
	
	std::vector<HE_Edge*> result;	
	while(it != copy_edges.end()) {
		e = (*it);
		HE_Face* f1 = e->getFirstFace();
		HE_Face* f2 = e->getSecondFace();
		if( (f1 == NULL) || (f2 == NULL) || (!containsFace(f1)) || (!constainsFace(f2)) {
			result.push_back(e);
		}
	}
	return result;

}