Re: [PATCH vdagent 1/2] vdagent-x11-randr: Fix building warning

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

Thanks, applied minus the pointer casts. I'll write a better fix for that myself.

Regards,

Hans


On 12/05/2012 09:01 AM, Dunrong Huang wrote:
This patch can be safely applied since it just deletes unused variables.

Signed-off-by: Dunrong Huang <riegamaths@xxxxxxxxx>
---
  src/vdagent-x11-randr.c | 8 +-------
  1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/src/vdagent-x11-randr.c b/src/vdagent-x11-randr.c
index 2c1819a..e171e7a 100644
--- a/src/vdagent-x11-randr.c
+++ b/src/vdagent-x11-randr.c
@@ -204,9 +204,6 @@ static void delete_mode(struct vdagent_x11 *x11, int output_index, const char* n
      int m;
      XRRModeInfo *mode;
      XRROutputInfo *output_info;
-    XRRCrtcInfo *crtc_info;
-    RRCrtc crtc;
-    int current_mode = -1;

      output_info = x11->randr.outputs[output_index];
      if (output_info->ncrtc != 1) {
@@ -214,9 +211,6 @@ static void delete_mode(struct vdagent_x11 *x11, int output_index, const char* n
                 "failed to delete mode", output_info->ncrtc);
          return;
      }
-    crtc_info = crtc_from_id(x11, output_info->crtcs[0]);
-    current_mode = crtc_info->mode;
-    crtc = output_info->crtc;
      for (m = 0 ; m < x11->randr.res->nmode; ++m) {
          mode = &x11->randr.res->modes[m];
          if (strcmp(mode->name, name) == 0)
@@ -689,7 +683,7 @@ void vdagent_x11_set_monitor_config(struct vdagent_x11 *x11,
          if (!xrandr_add_and_set(x11, i, x, y, width, height) &&
                  mon_config->num_of_monitors == 1) {
              set_screen_to_best_size(x11, width, height,
-                                    &primary_w, &primary_h);
+                                    (int *)&primary_w, (int *)&primary_h);
              goto update;
          }
      }

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel


[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]