DistRen Changeset - 249d80ebb87a · protofusion repositories

Changeset - 249d80ebb87a

Parent rev.

Child rev.

[Not reviewed]

default

0 1 0

LordOfWar - 16 years ago 2009-10-09 13:52:38

createQueueFromXML() now cleans up after itself like a good little function

1 file changed with 5 insertions and 5 deletions:

src/server/distrend.c

0 comments (0 inline, 0 general)

src/server/distrend.c

➞

Show inline comments

@@ @@ -522,7 +522,7 @@ int updateJobListXML(struct distrenjob * @@
   _distren_asprintf(&tmp, "job_list.xml");
   writer = xmlNewTextWriterFilename(tmp, 0);
-  xmlTextWriterStartDocument(writer, NULL, "UTF-8", NULL);
+  xmlTextWriterStartDocument(writer, NULL, "utf-8", NULL);
   // create root element job_list
   xmlTextWriterStartElement(writer, (xmlChar*)"job_list");
@@ @@ -552,7 +552,7 @@ int updateJobListXML(struct distrenjob * @@
 // and then adds the jobs to the queue, as if it were never shut down
 int createQueueFromXML(struct distrenjob *head)
+{
-  xmlChar *tmp2;
+  xmlChar *tmp;
   xmlDocPtr doc;  // holds xml document in memory
   xmlNodePtr cur; // points to the current xml element node
@@ @@ -583,16 +583,16 @@ int createQueueFromXML(struct distrenjob @@
   // scans the list of all jobs that were in queue before DistRen shutdown
   while(cur != NULL)
+    {
       tmp2 = xmlNodeListGetString(doc, cur->xmlChildrenNode, 1);
       fprintf(stderr, "adding job: %s", tmp2);
       tmp = xmlNodeListGetString(doc, cur->xmlChildrenNode, 1);
       fprintf(stderr, "adding job: %s", tmp);
       // add job from job number @TODO create function with parameters (struct distrenjob *head, int job_number)
       // LordOfWar calls dibs on above todo
       xmlFree(tmp);
       cur = cur->next; // go to next child element of job_list element
+    }
   xmlFreeDoc(doc);
   xmlFree(tmp2);
   return 1;
+}

0 comments (0 inline, 0 general)