Re: STL: Could you make this snippet more efficient

[..]
set< unsigned int > modpckset; // has to be defined outside the loop
string line;
while ( getline( fp, line ) ) // proper way to read until end of file
{
  istringstream ss( line );
  copy( istream_iterator< unsigned int >( ss ),
     istream_iterator< unsigned int >(),
     inserter( modpckset, modpckset.begin() ) );
}

With the above code, getline will read in a line of the file at a
time, then parse it out to a bunch of unsigned ints. We don't need to
do the extra step:

  set< unsigned int > modpckset;
  copy( istream_iterator< unsigned int >( fp ),
     istream_iterator< unsigned int >(),
     inserter( modpckset, modpckset.begin() ) );

The above will do the same thing.

[..]