vector assign() trouble

[nandor.sieben]

I am trying to replace

template < class T void
set2vector (const set < T &s, vector < T &v)
{
typename set < T >::iterator it;
for (it = s.begin (); it != s.end (); it++) {
v.push_back (*it);
}
}

with

template < class T void
set2vector (const set < T &s, vector < T &v)
{
v.assign(BE(s)) ;
}

but I get a segmentation fault. I am using g++ on Fedora core 4. Aren't
these code segments equivalent?

[Victor Bazarov]

Roland Pibinger wrote:

On Mon, 8 Jan 2007 20:10:54 -0500, "Victor Bazarov" wrote:

>I would guess that the function is for repeated use with already
created vector (no need to reconstruct it). The following does the
same thing

existingvector. swap(std::vecto r(s.begin(), s.end()));

You cannot use a temporary with vector::swap().

Right. It ought to be the other way around:

std::vector(s.b egin(), s.end()).swap(e xistingvector);

V
--
Please remove capital 'A's when replying by e-mail
I do not respond to top-posted replies, please don't ask

[peter koch]

Sylvester Hesp skrev:

"peter koch" <pe************ ***@gmail.comwr ote in message
news:11******** *************@1 1g2000cwr.googl egroups.com...

As Victor already pointed out, assign assigns and does not create new
elements.

I'm sorry but that's just nonsense. The assign function does the same as
it's corresponding constructor in vector, namelijk reconstructing the vector
with the range of elements you've provided.

[snip]

Right. One of the (few) times that Victor was wrong and my appeal to
authority became to strong;-)

/Peter