Hopefully fixed wrong libxml2 usage

We did a xmlCleanupParser () call everytime we finished parsing; this is wrong (http://xmlsoft.org/html/libxml-parser.html#xmlCleanupParser)

2 files changed, 6 insertions, 7 deletions

diff --git a/libpiano/main.c b/libpiano/main.c
index e475b59..cc4e65a 100644
--- a/libpiano/main.c
+++ b/libpiano/main.c
@@ -32,7 +32,9 @@ THE SOFTWARE.
 #include "xml.h"
 #include "crypt.h"
 
-/*	initialize piano handle, set up curl handle and settings
+/*	initialize piano handle, set up curl handle and settings; note: you
+ *	_must_ init curl and libxml2 using curl_global_init (CURL_GLOBAL_SSL)
+ *	and xmlInitParser (), you also _must_ cleanup their garbage on your own!
  *	@author PromyLOPh
  *	@added 2008-06-05
  *	@param piano handle
@@ -140,7 +142,9 @@ void PianoDestroyPlaylist (PianoHandle_t *ph) {
 	ph->playlist = NULL;
 }
 
-/*	frees the whole piano handle structure
+/*	frees the whole piano handle structure; this will _not_ cleanup curl's
+ *	internal garbage, you have to call curl_global_cleanup () and
+ *	xmlCleanupParser () for libxml2
  *	@author PromyLOPh
  *	@added 2008-06-05
  *	@param piano handle
diff --git a/libpiano/xml.c b/libpiano/xml.c
index 048f792..6f8cb55 100644
--- a/libpiano/xml.c
+++ b/libpiano/xml.c
@@ -203,7 +203,6 @@ void PianoXmlParseUserinfo (PianoHandle_t *ph, char *xml) {
 	PianoXmlStructParser (structRoot, PianoXmlParseUserinfoCb, &ph->user);
 
 	xmlFreeDoc (doc);
-	xmlCleanupParser();
 }
 
 /*	parse stations returned by pandora
@@ -244,7 +243,6 @@ void PianoXmlParseStations (PianoHandle_t *ph, char *xml) {
 	}
 
 	xmlFreeDoc (doc);
-	xmlCleanupParser();
 }
 
 /*	parses playlist; used when searching too
@@ -284,7 +282,6 @@ void PianoXmlParsePlaylist (PianoHandle_t *ph, char *xml) {
 	}
 
 	xmlFreeDoc (doc);
-	xmlCleanupParser();
 }
 
 /*	parse simple answers like this: <?xml version="1.0" encoding="UTF-8"?>
@@ -310,7 +307,6 @@ PianoReturn_t PianoXmlParseSimple (char *xml) {
 	}
 
 	xmlFreeDoc (doc);
-	xmlCleanupParser();
 
 	return ret;
 }
@@ -413,7 +409,6 @@ void PianoXmlParseSearch (char *searchXml,
 	PianoXmlStructParser (structRoot, PianoXmlParseSearchCb, searchResult);
 
 	xmlFreeDoc (doc);
-	xmlCleanupParser();
 }
 
 /*	encode reserved xml chars