postprocess: simplify pp_free() code
authorBrian Paul <brianp@vmware.com>
Sat, 16 Nov 2013 20:55:40 +0000 (13:55 -0700)
committerBrian Paul <brianp@vmware.com>
Mon, 18 Nov 2013 15:56:34 +0000 (08:56 -0700)
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
src/gallium/auxiliary/postprocess/pp_init.c

index a49a23d11d4a3e6f518b34446e130b28a8e4dcfa..bbebb5ce6e69343970ff5bfb8ff4a12c3d04c40f 100644 (file)
@@ -169,11 +169,12 @@ pp_free(struct pp_queue_t *ppq)
 {
    unsigned int i, j;
 
-   if (ppq) {
-      pp_free_fbos(ppq);
-   }
+   if (!ppq)
+      return;
 
-   if (ppq && ppq->p) {
+   pp_free_fbos(ppq);
+
+   if (ppq->p) {
       if (ppq->p->pipe && ppq->filters && ppq->shaders) {
          for (i = 0; i < ppq->n_filters; i++) {
             unsigned int filter = ppq->filters[i];
@@ -221,17 +222,15 @@ pp_free(struct pp_queue_t *ppq)
       FREE(ppq->p);
    }
 
-   if (ppq) {
-      /*
-       * Handle partial initialization for common resource destruction
-       * in the create path.
-       */
-      FREE(ppq->filters);
-      FREE(ppq->shaders);
-      FREE(ppq->pp_queue);
+   /*
+    * Handle partial initialization for common resource destruction
+    * in the create path.
+    */
+   FREE(ppq->filters);
+   FREE(ppq->shaders);
+   FREE(ppq->pp_queue);
   
-      FREE(ppq);
-   }
+   FREE(ppq);
 
    pp_debug("Queue taken down.\n");
 }