a bug in g++ std::map implementation?

[Fei Liu]

Hello, I am getting something really wierd out of this source code, can
you please verify if you also get segmentation fault (memory failure)?

http_form.h: (sorry about the line wrapping, email client problem)
#ifndef HTTP_FORM_H
#define HTTP_FORM_H

#include <map>
#include <string>
#include <iostream>

namespace feiliu{
template <typename value_type>
class http_form{
public:
http_form();
~http_form();
void add_entry(const std::pair<std:: string, value_type&
entry){
entries.insert( entry);
}
void add_entry(const std::string & key, const value_type &
value){
entries.insert( std::pair<std:: string, value_type>(key ,
value));
}
std::string get_form_data() ;
private:
std::map<std::s tring, value_typeentri es;
};

template <typename value_type>
std::string http_form<value _type>::get_for m_data(){
typename std::map<std::s tring, value_type>::it erator iter =
entries.begin() ;
std::cout << "size of map: " << entries.size() << '\n';
for(int i = 0; i < entries.size(); i ++){
std::cout << "key = " << iter->first << " value = " <<
iter->second << '\n';
++ iter;
}
for(iter = entries.begin() ; iter != entries.end(); ++ iter)
std::cout << "key = " << iter->first << " value = " <<
iter->second << '\n';
}
}

#endif

http_form.cpp:
#include "http_form. h"
template <typename value_type>
feiliu::http_fo rm<value_type>: :http_form(){
}

//std::string http_form::get_ form_data(){
template <typename value_type>
feiliu::http_fo rm<value_type>: :~http_form(){
entries.clear() ;
}

#ifdef TEST_HTTP_FORM
using std::string;
int main(){
feiliu::http_fo rm<stringform;
form.add_entry( "hello", "world");
form.add_entry( "foo", "bar");
form.get_form_d ata();
}
#endif

compile: g++ -DTEST_HTTP_FORM -g -O0 -o tf http_form.cpp
run: ./tf
size of map: 2
key = foo value = bar
key = hello value = world
key = foo value = bar
key = hello value = world
Segmentation fault
Apparently, there is something wrong with iterator using begin() and
end()...Any help is appreciated. Thanks,

Fei

[Alf P. Steinbach]

* Fei Liu:

>
compile: g++ -DTEST_HTTP_FORM -g -O0 -o tf http_form.cpp

Add options to up the warning level. Presumably the compiler will then
complain about a missing return statement.

--
A: Because it messes up the order in which people normally read text.
Q: Why is it such a bad thing?
A: Top-posting.
Q: What is the most annoying thing on usenet and in e-mail?

[Fei Liu]

Alf P. Steinbach wrote:

* Fei Liu:

compile: g++ -DTEST_HTTP_FORM -g -O0 -o tf http_form.cpp

Add options to up the warning level. Presumably the compiler will then
complain about a missing return statement.

Ah, that did the trick, adding return "" stopped the compiler from
generating bad assembly. Thanks,

Fei