[Xfce4-commits] <xfce4-settings:xrandr-display-settings> Duplicate memory instead of looping

[Jérôme Guelfucci]

Updating branch refs/heads/xrandr-display-settings
         to b7820b6f6e13ad6ea961d4506d36aada081c8ace (commit)
       from d5820ae44363a09beb82b0995d10ca5bfb9f8c06 (commit)

commit b7820b6f6e13ad6ea961d4506d36aada081c8ace
Author: Lionel Le Folgoc <mrpouit at gmail.com>
Date:   Sun Jun 27 11:09:46 2010 +0200

    Duplicate memory instead of looping

 xfce4-settings-helper/displays.c |   19 ++++++++-----------
 1 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/xfce4-settings-helper/displays.c b/xfce4-settings-helper/displays.c
index 9b0b404..3e4bde8 100644
--- a/xfce4-settings-helper/displays.c
+++ b/xfce4-settings-helper/displays.c
@@ -227,7 +227,7 @@ xfce_displays_helper_list_crtcs (Display            *xdisplay,
 {
     XfceRRCrtc  *crtcs;
     XRRCrtcInfo *crtc_info;
-    gint         n, m;
+    gint         n;
 
     g_return_val_if_fail (xdisplay != NULL, NULL);
     g_return_val_if_fail (resources != NULL, NULL);
@@ -243,22 +243,19 @@ xfce_displays_helper_list_crtcs (Display            *xdisplay,
         crtcs[n].rotations = crtc_info->rotations;
         crtcs[n].x = crtc_info->x;
         crtcs[n].y = crtc_info->y;
+
         crtcs[n].noutput = crtc_info->noutput;
         crtcs[n].outputs = NULL;
         if (crtc_info->noutput > 0)
-        {
-            crtcs[n].outputs = g_new0 (RROutput, crtc_info->noutput);
-            for (m = 0; m < crtc_info->noutput; ++m)
-                crtcs[n].outputs[m] = crtc_info->outputs[m];
-        }
+            crtcs[n].outputs = g_memdup (crtc_info->outputs,
+                                         crtc_info->noutput * sizeof (RROutput));
+
         crtcs[n].npossible = crtc_info->npossible;
         crtcs[n].possible = NULL;
         if (crtc_info->npossible > 0)
-        {
-            crtcs[n].possible = g_new0 (RROutput, crtc_info->npossible);
-            for (m = 0; m < crtc_info->npossible; ++m)
-                crtcs[n].possible[m] = crtc_info->possible[m];
-        }
+            crtcs[n].possible = g_memdup (crtc_info->possible,
+                                          crtc_info->npossible * sizeof (RROutput));
+
         crtcs[n].processed = FALSE;
         XRRFreeCrtcInfo (crtc_info);
     }