now descend from CamelMedium.

1999-08-25  bertrand  <Bertrand.Guiheneuf@aful.org>

	* camel/camel-mime-part.c: now descend from CamelMedium.

	* tests/test1.c (main): all headers must be strdup'ed
	(main): unref created objects

svn path=/trunk/; revision=1143

6 files changed, 141 insertions, 170 deletions

diff --git a/ChangeLog b/ChangeLog
index 885af8ca7a..a198da2fd5 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,10 @@
 1999-08-25  bertrand  <Bertrand.Guiheneuf@aful.org>
 
+	* camel/camel-mime-part.c: now descend from CamelMedium.
+
+	* tests/test1.c (main): all headers must be strdup'ed
+	(main): unref created objects
+
 	* camel/camel-medium.c (_set_content_object): 
 	(_get_content_object): these methods are 
 	in CamelMedium now.
diff --git a/camel/camel-medium.c b/camel/camel-medium.c
index b6a3ffc9fc..a8d7103237 100644
--- a/camel/camel-medium.c
+++ b/camel/camel-medium.c
@@ -61,6 +61,9 @@ camel_medium_class_init (CamelMediumClass *camel_medium_class)
 	camel_medium_class->remove_header = _remove_header;
 	camel_medium_class->get_header = _get_header;
 	
+	camel_medium_class->set_content_object = _set_content_object;
+	camel_medium_class->get_content_object = _get_content_object;
+	
 	
 	
 	/* virtual method overload */
@@ -76,6 +79,7 @@ camel_medium_init (gpointer   object,  gpointer   klass)
 	CamelMedium *camel_medium = CAMEL_MEDIUM (object);
 	
 	camel_medium->headers =  g_hash_table_new (g_str_hash, g_str_equal);
+	camel_medium->content =  NULL;
 }
 
 
@@ -115,6 +119,7 @@ _finalize (GtkObject *object)
 	CAMEL_LOG_FULL_DEBUG ("Entering CamelMedium::finalize\n");
 
 	if (medium->headers) {
+#warning Free hash table elements
 		g_hash_table_destroy (medium->headers);
 	}
 
diff --git a/camel/camel-mime-message.c b/camel/camel-mime-message.c
index e91b5a3624..69b2ca7e8c 100644
--- a/camel/camel-mime-message.c
+++ b/camel/camel-mime-message.c
@@ -143,6 +143,14 @@ camel_mime_message_init (gpointer object, gpointer klass)
 	
 	camel_mime_message->recipients =  g_hash_table_new (g_strcase_hash, g_strcase_equal);
 	camel_mime_message->flags = g_hash_table_new (g_strcase_hash, g_strcase_equal);
+	
+	camel_mime_message->received_date = NULL;
+	camel_mime_message->sent_date = NULL;
+	camel_mime_message->subject = NULL;
+	camel_mime_message->reply_to = NULL;
+	camel_mime_message->from = NULL;
+	camel_mime_message->folder = NULL;
+	camel_mime_message->session = NULL;
 }
 
 GtkType
@@ -189,7 +197,6 @@ _finalize (GtkObject *object)
 	if (message->flags)
 		g_hash_table_foreach (message->flags, g_hash_table_generic_free, NULL);
 	
-
 	GTK_OBJECT_CLASS (parent_class)->finalize (object);
 	CAMEL_LOG_FULL_DEBUG ("Leaving CamelMimeMessage::finalize\n");
 }
diff --git a/camel/camel-mime-part.c b/camel/camel-mime-part.c
index 1620eb771f..1e41e5038d 100644
--- a/camel/camel-mime-part.c
+++ b/camel/camel-mime-part.c
@@ -46,14 +46,24 @@ typedef enum {
 static GHashTable *header_name_table;
 
 
-static CamelDataWrapperClass *parent_class=NULL;
+static CamelMediumClass *parent_class=NULL;
 
 /* Returns the class for a CamelMimePart */
 #define CMP_CLASS(so) CAMEL_MIME_PART_CLASS (GTK_OBJECT(so)->klass)
 
-static void _add_header (CamelMimePart *mime_part, gchar *header_name, gchar *header_value);
-static void _remove_header (CamelMimePart *mime_part, const gchar *header_name);
-static const gchar *_get_header (CamelMimePart *mime_part, const gchar *header_name);
+/* from GtkObject */
+static void _finalize (GtkObject *object);
+
+/* from CamelDataWrapper */
+static void _write_to_stream (CamelDataWrapper *data_wrapper, CamelStream *stream);
+void _construct_from_stream (CamelDataWrapper *data_wrapper, CamelStream *stream);
+
+/* from CamelMedia */ 
+static void _add_header (CamelMedium *medium, gchar *header_name, gchar *header_value);
+
+static void _set_content_object (CamelMedium *medium, CamelDataWrapper *content);
+
+/* CamelMimePart methods */
 static void _set_description (CamelMimePart *mime_part, const gchar *description);
 static const gchar *_get_description (CamelMimePart *mime_part);
 static void _set_disposition (CamelMimePart *mime_part, const gchar *disposition);
@@ -73,14 +83,9 @@ static const GList *_get_header_lines (CamelMimePart *mime_part);
 static void _set_content_type (CamelMimePart *mime_part, const gchar *content_type);
 static GMimeContentField  *_get_content_type (CamelMimePart *mime_part);
 
-static const CamelDataWrapper *_get_content_object(CamelMimePart *mime_part);
-static void _set_content_object(CamelMimePart *mime_part, CamelDataWrapper *content);
-
-static void _write_to_stream (CamelDataWrapper *data_wrapper, CamelStream *stream);
 static gboolean _parse_header_pair (CamelMimePart *mime_part, gchar *header_name, gchar *header_value);
-void _construct_from_stream (CamelDataWrapper *data_wrapper, CamelStream *stream);
 
-static void _finalize (GtkObject *object);
+
 
 /* loads in a hash table the set of header names we */
 /* recognize and associate them with a unique enum  */
@@ -101,16 +106,14 @@ _init_header_name_table()
 static void
 camel_mime_part_class_init (CamelMimePartClass *camel_mime_part_class)
 {
+	CamelMediumClass *camel_medium_class = CAMEL_MEDIUM_CLASS (camel_mime_part_class);
 	CamelDataWrapperClass *camel_data_wrapper_class = CAMEL_DATA_WRAPPER_CLASS (camel_mime_part_class);
 	GtkObjectClass *gtk_object_class = GTK_OBJECT_CLASS (camel_data_wrapper_class);
 
-	parent_class = gtk_type_class (camel_data_wrapper_get_type ());
+	parent_class = gtk_type_class (camel_medium_get_type ());
 	_init_header_name_table();
 	
 	/* virtual method definition */
-	camel_mime_part_class->add_header = _add_header;
-	camel_mime_part_class->remove_header = _remove_header;
-	camel_mime_part_class->get_header = _get_header;
 	camel_mime_part_class->set_description = _set_description;
 	camel_mime_part_class->get_description = _get_description;
 	camel_mime_part_class->set_disposition = _set_disposition;
@@ -131,12 +134,11 @@ camel_mime_part_class_init (CamelMimePartClass *camel_mime_part_class)
 	camel_mime_part_class->get_content_type = _get_content_type;
 	
 	camel_mime_part_class->parse_header_pair = _parse_header_pair;
-	camel_mime_part_class->get_content_object = _get_content_object;
-	camel_mime_part_class->set_content_object = _set_content_object;
-	
-	
-	
-	/* virtual method overload */
+
+	/* virtual method overload */	
+	camel_medium_class->add_header = _add_header;
+	camel_medium_class->set_content_object = _set_content_object;
+
 	camel_data_wrapper_class->write_to_stream = _write_to_stream;
 	camel_data_wrapper_class->construct_from_stream = _construct_from_stream;
 
@@ -148,7 +150,6 @@ camel_mime_part_init (gpointer   object,  gpointer   klass)
 {
 	CamelMimePart *camel_mime_part = CAMEL_MIME_PART (object);
 	
-	camel_mime_part->headers =  g_hash_table_new (g_str_hash, g_str_equal);
 	camel_mime_part->content_type = gmime_content_field_new (NULL, NULL);
 	camel_mime_part->description = NULL;
 	camel_mime_part->disposition = NULL;
@@ -158,7 +159,6 @@ camel_mime_part_init (gpointer   object,  gpointer   klass)
 	camel_mime_part->encoding = NULL;
 	camel_mime_part->filename = NULL;
 	camel_mime_part->header_lines = NULL;
-	camel_mime_part->content = NULL;
 }
 
 
@@ -182,7 +182,7 @@ camel_mime_part_get_type (void)
 			(GtkClassInitFunc) NULL,
 		};
 		
-		camel_mime_part_type = gtk_type_unique (camel_data_wrapper_get_type (), &camel_mime_part_info);
+		camel_mime_part_type = gtk_type_unique (camel_medium_get_type (), &camel_mime_part_info);
 	}
 	
 	return camel_mime_part_type;
@@ -198,10 +198,7 @@ _finalize (GtkObject *object)
 
 	CAMEL_LOG_FULL_DEBUG ("Entering CamelMimePart::finalize\n");
 
-	if (mime_part->headers) {
-#warning Free hash table elements
-		g_hash_table_destroy (mime_part->headers);
-	}
+
 
 	if (mime_part->description) g_free (mime_part->description);
 	if (mime_part->disposition) gmime_content_field_unref (mime_part->disposition);
@@ -213,20 +210,18 @@ _finalize (GtkObject *object)
 	if (mime_part->header_lines) string_list_free (mime_part->header_lines);
 
 	if (mime_part->content_type) gmime_content_field_unref (mime_part->content_type);
-	if (mime_part->content) {
-		CAMEL_LOG_FULL_DEBUG ("CamelMimePart::finalize, unreferencing content object\n");
-		CAMEL_LOG_FULL_DEBUG ("CamelMimePart::finalize,Destroying  object %p\n", mime_part->content);
-		CAMEL_LOG_FULL_DEBUG ("CamelMimePart::finalize, object type : %s\n", gtk_type_name (GTK_OBJECT_TYPE (mime_part->content)));
-		gtk_object_unref (GTK_OBJECT (mime_part->content));
-	}
+
 	GTK_OBJECT_CLASS (parent_class)->finalize (object);
 	CAMEL_LOG_FULL_DEBUG ("Leaving CamelMimePart::finalize\n");
 }
 
 
+/* **** */
+
 static void
-_add_header (CamelMimePart *mime_part, gchar *header_name, gchar *header_value)
+_add_header (CamelMedium *medium, gchar *header_name, gchar *header_value)
 {
+	CamelMimePart *mime_part = CAMEL_MIME_PART (medium);
 	gboolean header_exists;
 	gchar *old_header_name;
 	gchar *old_header_value;
@@ -234,73 +229,12 @@ _add_header (CamelMimePart *mime_part, gchar *header_name, gchar *header_value)
 	/* Try to parse the header pair. If it corresponds to something   */
 	/* known, the job is done in the parsing routine. If not,         */
 	/* we simply add the header in a raw fashion                      */
-	if (CMP_CLASS(mime_part)->parse_header_pair (mime_part, header_name, header_value)) 
-		return;
-	header_exists = g_hash_table_lookup_extended (mime_part->headers, header_name, 
-						      (gpointer *) &old_header_name,
-						      (gpointer *) &old_header_value);
-	if (header_exists) {
-		g_free (old_header_name);
-		g_free (old_header_value);
-	}
-	
-	g_hash_table_insert (mime_part->headers, header_name, header_value);
+	if (! CMP_CLASS(mime_part)->parse_header_pair (mime_part, header_name, header_value) ) 		
+		parent_class->add_header (medium, header_name, header_value);
 }
 
 
-void
-camel_mime_part_add_header (CamelMimePart *mime_part, gchar *header_name, gchar *header_value)
-{
-	CMP_CLASS(mime_part)->add_header(mime_part, header_name, header_value);
-}
-
-
-
-static void
-_remove_header (CamelMimePart *mime_part, const gchar *header_name)
-{
-	
-	gboolean header_exists;
-	gchar *old_header_name;
-	gchar *old_header_value;
-	
-	header_exists = g_hash_table_lookup_extended (mime_part->headers, header_name, 
-						      (gpointer *) &old_header_name,
-						      (gpointer *) &old_header_value);
-	if (header_exists) {
-		g_free (old_header_name);
-		g_free (old_header_value);
-	}
-	
-	g_hash_table_remove (mime_part->headers, header_name);
-	
-}
-
-void
-camel_mime_part_remove_header (CamelMimePart *mime_part, const gchar *header_name)
-{
-	CMP_CLASS(mime_part)->remove_header(mime_part, header_name);
-}
-
-
-
-static const gchar *
-_get_header (CamelMimePart *mime_part, const gchar *header_name)
-{
-	
-	gchar *old_header_name;
-	gchar *old_header_value;
-	gchar *header_value;
-	
-	header_value = (gchar *)g_hash_table_lookup (mime_part->headers, header_name);
-	return header_value;
-}
 
-const gchar *
-camel_mime_part_get_header (CamelMimePart *mime_part, const gchar *header_name)
-{
-	return CMP_CLASS(mime_part)->get_header (mime_part, header_name);
-}
 
 
 
@@ -319,6 +253,9 @@ camel_mime_part_set_description (CamelMimePart *mime_part, const gchar *descript
 
 
 
+/* **** */
+
+
 
 static const gchar *
 _get_description (CamelMimePart *mime_part)
@@ -334,6 +271,9 @@ camel_mime_part_get_description (CamelMimePart *mime_part)
 
 
 
+/* **** */
+
+
 static void
 _set_disposition (CamelMimePart *mime_part, const gchar *disposition)
 {
@@ -353,6 +293,9 @@ camel_mime_part_set_disposition (CamelMimePart *mime_part, const gchar *disposit
 }
 
 
+/* **** */
+
+
 
 static const gchar *
 _get_disposition (CamelMimePart *mime_part)
@@ -386,6 +329,9 @@ camel_mime_part_set_filename (CamelMimePart *mime_part, gchar *filename)
 
 
 
+/* **** */
+
+
 static const gchar *
 _get_filename (CamelMimePart *mime_part)
 {
@@ -400,6 +346,9 @@ camel_mime_part_get_filename (CamelMimePart *mime_part)
 }
 
 
+/* **** */
+
+
 /* this routine must not be public */
 static void
 _set_content_id (CamelMimePart *mime_part, gchar *content_id)
@@ -416,6 +365,9 @@ _get_content_id (CamelMimePart *mime_part)
 }
 
 
+/* **** */
+
+
 const gchar *
 camel_mime_part_get_content_id (CamelMimePart *mime_part)
 {
@@ -432,6 +384,9 @@ _set_content_MD5 (CamelMimePart *mime_part, gchar *content_MD5)
 }
 
 
+/* **** */
+
+
 static const gchar *
 _get_content_MD5 (CamelMimePart *mime_part)
 {
@@ -445,6 +400,9 @@ camel_mime_part_get_content_MD5 (CamelMimePart *mime_part)
 }
 
 
+/* **** */
+
+
 
 static void
 _set_encoding (CamelMimePart *mime_part, gchar *encoding)
@@ -460,6 +418,9 @@ camel_mime_part_set_encoding (CamelMimePart *mime_part, gchar *encoding)
 }
 
 
+/* **** */
+
+
 
 static const gchar *
 _get_encoding (CamelMimePart *mime_part)
@@ -475,6 +436,9 @@ camel_mime_part_get_encoding (CamelMimePart *mime_part)
 
 
 
+/* **** */
+
+
 
 static void
 _set_content_languages (CamelMimePart *mime_part, GList *content_languages)
@@ -490,6 +454,9 @@ camel_mime_part_set_content_languages (CamelMimePart *mime_part, GList *content_
 }
 
 
+/* **** */
+
+
 
 static const GList *
 _get_content_languages (CamelMimePart *mime_part)
@@ -505,6 +472,9 @@ camel_mime_part_get_content_languages (CamelMimePart *mime_part)
 }
 
 
+/* **** */
+
+
 
 static void
 _set_header_lines (CamelMimePart *mime_part, GList *header_lines)
@@ -520,6 +490,9 @@ camel_mime_part_set_header_lines (CamelMimePart *mime_part, GList *header_lines)
 }
 
 
+/* **** */
+
+
 
 static const GList *
 _get_header_lines (CamelMimePart *mime_part)
@@ -536,8 +509,10 @@ camel_mime_part_get_header_lines (CamelMimePart *mime_part)
 }
 
 
+/* **** */
+
+
 
-/*********/
 static void
 _set_content_type (CamelMimePart *mime_part, const gchar *content_type)
 {
@@ -551,6 +526,9 @@ camel_mime_part_set_content_type (CamelMimePart *mime_part, gchar *content_type)
 	CMP_CLASS(mime_part)->set_content_type (mime_part, content_type);
 }
 
+/* **** */
+
+
 static GMimeContentField *
 _get_content_type (CamelMimePart *mime_part)
 {
@@ -566,35 +544,15 @@ camel_mime_part_get_content_type (CamelMimePart *mime_part)
 /*********/
 
 
-static const CamelDataWrapper *
-_get_content_object(CamelMimePart *mime_part)
-{
-	return mime_part->content;
-	
-}
-
-
-const CamelDataWrapper *
-camel_mime_part_get_content_object(CamelMimePart *mime_part)
-{
-	return CMP_CLASS(mime_part)->get_content_object (mime_part);
-}
-
-
-
 
 static void
-_set_content_object(CamelMimePart *mime_part, CamelDataWrapper *content)
+_set_content_object (CamelMedium *medium, CamelDataWrapper *content)
 {
+	CamelMimePart *mime_part = CAMEL_MIME_PART (medium);
 	GMimeContentField *object_content_field;
 
-	CAMEL_LOG_FULL_DEBUG ("Entering CamelMimePart::set_content_object\n");
-	if (mime_part->content) {
-		CAMEL_LOG_FULL_DEBUG ("CamelMimePart::set_content_object unreferencing old content object\n");
-		gtk_object_unref (GTK_OBJECT (mime_part->content));
-	}
-	gtk_object_ref (GTK_OBJECT (content));
-	mime_part->content = content;
+	parent_class->set_content_object (medium, content);
+
 	object_content_field = camel_data_wrapper_get_mime_type_field (content);
 	if (mime_part->content_type && (mime_part->content_type != object_content_field)) 
 		gmime_content_field_unref (mime_part->content_type);
@@ -604,11 +562,7 @@ _set_content_object(CamelMimePart *mime_part, CamelDataWrapper *content)
 	
 }
 
-void 
-camel_mime_part_set_content_object(CamelMimePart *mime_part, CamelDataWrapper *content)
-{
-	CMP_CLASS(mime_part)->set_content_object (mime_part, content);
-}
+/* **** */
 
 
 
@@ -623,11 +577,12 @@ camel_mime_part_set_content_object(CamelMimePart *mime_part, CamelDataWrapper *c
 static void
 _write_content_to_stream (CamelMimePart *mime_part, CamelStream *stream)
 {
+	CamelMedium *medium = CAMEL_MEDIUM (mime_part);
 	guint buffer_size;
 	gchar *buffer;
 	gchar *encoded_buffer;
 
-	CamelDataWrapper *content = mime_part->content;
+	CamelDataWrapper *content = medium->content;
 	CAMEL_LOG_FULL_DEBUG ( "Entering CamelMimePart::_write_content_to_stream\n");
 	CAMEL_LOG_FULL_DEBUG ( "CamelMimePart::_write_content_to_stream, content=%p\n", content);
 	if (!content) return;
@@ -656,7 +611,8 @@ static void
 _write_to_stream (CamelDataWrapper *data_wrapper, CamelStream *stream)
 {
 	CamelMimePart *mp = CAMEL_MIME_PART (data_wrapper);
-	
+	CamelMedium *medium = CAMEL_MEDIUM (data_wrapper);
+
 	CAMEL_LOG_FULL_DEBUG ( "Entering CamelMimePart::write_to_stream\n");
 	
 	CAMEL_LOG_FULL_DEBUG ( "CamelMimePart::write_to_stream writing content-disposition\n");
@@ -673,7 +629,7 @@ _write_to_stream (CamelDataWrapper *data_wrapper, CamelStream *stream)
 	write_header_with_glist_to_stream (stream, "Content-Language", mp->content_languages,", ");
 	
 	CAMEL_LOG_FULL_DEBUG ( "CamelMimePart::write_to_stream writing other headers\n");
-	write_header_table_to_stream (stream, mp->headers);
+	write_header_table_to_stream (stream, medium->headers);
 	
 	CAMEL_LOG_FULL_DEBUG ( "CamelMimePart::write_to_stream writing content-type\n");
 	gmime_content_field_write_to_stream (mp->content_type, stream);
@@ -779,7 +735,8 @@ _parse_hash_table_pair (gpointer key, gpointer value, gpointer user_data)
 	if  (header_value)
 		CAMEL_LOG_FULL_DEBUG ( "header value :%s\n", header_value);
 	
-	camel_mime_part_add_header (mime_part, header_name, header_value);
+	camel_medium_add_header ( CAMEL_MEDIUM (mime_part), header_name, header_value);
+
 	CAMEL_LOG_FULL_DEBUG ( "--------- End -----------------\n"); 
 }
 
@@ -822,7 +779,7 @@ _construct_from_stream (CamelDataWrapper *data_wrapper, CamelStream *stream)
 	g_free (mime_type);
 	content_object = CAMEL_DATA_WRAPPER (gtk_type_new (content_object_type));
 	camel_data_wrapper_set_mime_type_field (content_object, camel_mime_part_get_content_type (mime_part));
-	camel_mime_part_set_content_object (mime_part, content_object);
+	camel_medium_set_content_object ( CAMEL_MEDIUM (mime_part), content_object);
 	camel_data_wrapper_construct_from_stream (content_object, stream);
 
 	/* the object is referenced in the set_content_object method, so unref it here */
@@ -854,22 +811,24 @@ void
 camel_mime_part_set_text (CamelMimePart *camel_mime_part, gchar *text)
 {
 	CamelSimpleDataWrapper *simple_data_wrapper;
+	CamelMedium *medium = CAMEL_MEDIUM (camel_mime_part);
+
 	CAMEL_LOG_FULL_DEBUG ("CamelMimePart:: Entering set_text\n");
 	CAMEL_LOG_TRACE ("CamelMimePart::set_text, setting text as a mime part content\n");
-	if (camel_mime_part->content) {
+	if (medium->content) {
 		CAMEL_LOG_FULL_DEBUG ("CamelMimePart::set_text unreferencing old content object\n");
-		gtk_object_unref (GTK_OBJECT (camel_mime_part->content));
+		gtk_object_unref (GTK_OBJECT (medium->content));
 	}
 	if (text) {
 		simple_data_wrapper = camel_simple_data_wrapper_new ();
 		CAMEL_LOG_FULL_DEBUG ("CamelMimePart::set_text calling CamelSimpleDataWrapper:set_text with %d chars\n", strlen (text));
 		camel_data_wrapper_set_mime_type (CAMEL_DATA_WRAPPER (simple_data_wrapper), "text/plain");
 		camel_simple_data_wrapper_set_text ( simple_data_wrapper, text);
-		camel_mime_part_set_content_object (camel_mime_part, CAMEL_DATA_WRAPPER (simple_data_wrapper));
+		camel_medium_set_content_object ( CAMEL_MEDIUM (camel_mime_part), CAMEL_DATA_WRAPPER (simple_data_wrapper));
 		gtk_object_unref (GTK_OBJECT (simple_data_wrapper));
-	} else camel_mime_part->content = NULL;
-		
-		CAMEL_LOG_FULL_DEBUG ("CamelMimePart:: Leaving camel_mime_part_set_text\n");
+	} else medium->content = NULL;
+	
+	CAMEL_LOG_FULL_DEBUG ("CamelMimePart:: Leaving camel_mime_part_set_text\n");
 }
 
  
diff --git a/camel/camel-mime-part.h b/camel/camel-mime-part.h
index 3d74350645..10c8ca58a4 100644
--- a/camel/camel-mime-part.h
+++ b/camel/camel-mime-part.h
@@ -32,7 +32,7 @@ extern "C" {
 #endif /* __cplusplus }*/
 
 #include <gtk/gtk.h>
-#include "camel-media.h"
+#include "camel-medium.h"
 #include "camel-stream.h"
 
 
@@ -46,9 +46,8 @@ extern "C" {
    would regret it one day */
 typedef struct 
 {
-	CamelDataWrapper parent_object;
+	CamelMedium parent_object;
 	
-	GHashTable *headers;
 	gchar *description;
 	GMimeContentField *disposition;
 	gchar *content_id;
@@ -59,19 +58,15 @@ typedef struct
 	GList *header_lines;
 	
 	GMimeContentField *content_type;
-	CamelDataWrapper *content; /* part real content */
 	
 } CamelMimePart;
 
 
 
 typedef struct {
-	CamelDataWrapperClass parent_class;
+	CamelMediumClass parent_class;
 	
 	/* Virtual methods */	
-	void  (*add_header) (CamelMimePart *mime_part, gchar *header_name, gchar *header_value);
-	void  (*remove_header) (CamelMimePart *mime_part, const gchar *header_name);
-	const gchar * (*get_header) (CamelMimePart *mime_part, const gchar *header_name);
 	void  (*set_description) (CamelMimePart *mime_part, const gchar *description);
 	const gchar * (*get_description) (CamelMimePart *mime_part);
 	void  (*set_disposition) (CamelMimePart *mime_part, const gchar *disposition);
@@ -88,13 +83,12 @@ typedef struct {
 	const GList * (*get_content_languages) (CamelMimePart *mime_part);
 	void  (*set_header_lines) (CamelMimePart *mime_part, GList *header_lines);
 	const GList * (*get_header_lines) (CamelMimePart *mime_part);
+
 	void  (*set_content_type) (CamelMimePart *mime_part, const gchar *content_type);
 	GMimeContentField * (*get_content_type) (CamelMimePart *mime_part);
 	
 	gboolean (*parse_header_pair) (CamelMimePart *mime_part, gchar *header_name, gchar *header_value);
 	
-	const CamelDataWrapper * (*get_content_object) (CamelMimePart *mime_part);
-	void (*set_content_object) (CamelMimePart *mime_part, CamelDataWrapper *content);
 
 } CamelMimePartClass;
 
@@ -105,9 +99,6 @@ GtkType camel_mime_part_get_type (void);
 
 
 /* public methods */
-void camel_mime_part_add_header (CamelMimePart *mime_part, gchar *header_name, gchar *header_value);
-void camel_mime_part_remove_header (CamelMimePart *mime_part, const gchar *header_name);
-const gchar *camel_mime_part_get_header (CamelMimePart *mime_part, const gchar *header_name);
 void camel_mime_part_set_description (CamelMimePart *mime_part,	const gchar *description);
 const gchar *camel_mime_part_get_description (CamelMimePart *mime_part);
 void camel_mime_part_set_disposition (CamelMimePart *mime_part, const gchar *disposition);
@@ -123,9 +114,6 @@ const GList *camel_mime_part_get_content_languages (CamelMimePart *mime_part);
 void camel_mime_part_set_header_lines (CamelMimePart *mime_part, GList *header_lines);
 const GList *camel_mime_part_get_header_lines (CamelMimePart *mime_part);
 
-const CamelDataWrapper *camel_mime_part_get_content_object (CamelMimePart *mime_part);
-void camel_mime_part_set_content_object (CamelMimePart *mime_part, CamelDataWrapper *content);
-
 
 /* utility functions */
 void camel_mime_part_set_text (CamelMimePart *camel_mime_part, gchar *text);
diff --git a/tests/test1.c b/tests/test1.c
index ecf155fcdc..f5e3f164b6 100644
--- a/tests/test1.c
+++ b/tests/test1.c
@@ -20,30 +20,32 @@ main (int argc, char**argv)
 
 	gtk_init (&argc, &argv);
 	camel_init ();
-	message = camel_mime_message_new_with_session( (CamelSession *)NULL);
+	message = camel_mime_message_new_with_session ((CamelSession *)NULL);
+
 	camel_mime_part_set_description (CAMEL_MIME_PART (message), "a test");
-	camel_mime_part_add_header (CAMEL_MIME_PART (message), "X-test1", "the value of a test");
-	camel_mime_part_add_header (CAMEL_MIME_PART (message), "X-test2", "the value of another test");
+
+	camel_medium_add_header (CAMEL_MEDIUM (message), "X-test1", "the value of a test");
+	camel_medium_add_header (CAMEL_MEDIUM (message), "X-test2", "the value of another test");
 	/*camel_mime_part_add_content_language (CAMEL_MIME_PART (message), g_string_new ("es-ca"));*/
 
-	camel_mime_message_set_received_date (message, "Thu, 20 May 1999, 10:39:14 +0200");
-	camel_mime_message_set_subject (message, "A test message");
-	camel_mime_message_set_reply_to (message, "toto@toto.com");
-	camel_mime_message_set_from (message, "Bertrand.Guiheneuf@inria.fr");
+	camel_mime_message_set_received_date (message, g_strdup ("Thu, 20 May 1999, 10:39:14 +0200"));
+	camel_mime_message_set_subject (message, g_strdup ("A test message"));
+	camel_mime_message_set_reply_to (message, g_strdup ("toto@toto.com"));
+	camel_mime_message_set_from (message, g_strdup ("Bertrand.Guiheneuf@aful.org"));
 
-	camel_mime_message_add_recipient (message, RECIPIENT_TYPE_TO, "franck.dechamps@alseve.fr");
-	camel_mime_message_add_recipient (message, RECIPIENT_TYPE_TO, "mc@alseve.fr");
-	camel_mime_message_add_recipient (message, RECIPIENT_TYPE_TO, "richard.lengagne@inria.fr");
-	camel_mime_message_add_recipient (message, RECIPIENT_TYPE_CC, "Francois.fleuret@inria.fr");
-	camel_mime_message_add_recipient (message, RECIPIENT_TYPE_CC, "maury@justmagic.com");
- 	camel_mime_message_add_recipient (message, RECIPIENT_TYPE_BCC, "Bertrand.Guiheneuf@aful.org");
+	camel_mime_message_add_recipient (message, RECIPIENT_TYPE_TO, g_strdup ("franck.dechamps@alseve.fr"));
+	camel_mime_message_add_recipient (message, RECIPIENT_TYPE_TO, g_strdup ("mc@alseve.fr"));
+	camel_mime_message_add_recipient (message, RECIPIENT_TYPE_TO, g_strdup ("richard.lengagne@inria.fr"));
+	camel_mime_message_add_recipient (message, RECIPIENT_TYPE_CC, g_strdup ("Francois.fleuret@inria.fr"));
+	camel_mime_message_add_recipient (message, RECIPIENT_TYPE_CC, g_strdup ("maury@justmagic.com"));
+ 	camel_mime_message_add_recipient (message, RECIPIENT_TYPE_BCC, g_strdup ("Bertrand.Guiheneuf@aful.org"));
 
 	
 	multipart = camel_multipart_new ();
 	body_part = camel_mime_body_part_new ();
-	camel_mime_part_set_text (CAMEL_MIME_PART (body_part), "This is a test.\nThis is only a test.\n");
+	camel_mime_part_set_text (CAMEL_MIME_PART (body_part), g_strdup ("This is a test.\nThis is only a test.\n"));
 	camel_multipart_add_part (multipart, body_part);
-	camel_mime_part_set_content_object (CAMEL_MIME_PART (message), CAMEL_DATA_WRAPPER (multipart));
+	camel_medium_set_content_object (CAMEL_MEDIUM (message), CAMEL_DATA_WRAPPER (multipart));
 
 	stream = camel_stream_fs_new_with_name ("mail1.test", CAMEL_STREAM_FS_WRITE );
 	if (!stream)  {
@@ -53,6 +55,11 @@ main (int argc, char**argv)
 		       
 	camel_data_wrapper_write_to_stream (CAMEL_DATA_WRAPPER (message), stream);
 	camel_stream_close (stream);
+	gtk_object_unref (GTK_OBJECT (stream));
+
+	gtk_object_unref (GTK_OBJECT (message));
+	gtk_object_unref (GTK_OBJECT (multipart));
+	gtk_object_unref (GTK_OBJECT (body_part));
 	
 	printf ("Test1 finished\n");
 }