On 02/07/2012 12:21 AM, Peter Hutterer wrote:
xrandr uses "output", let's be consistent there.

Signed-off-by: Peter Hutterer<[email protected]>
---
  man/xinput.man  |    4 ++--
  src/transform.c |   16 ++++++++--------
  src/xinput.c    |    6 +++---
  src/xinput.h    |    2 +-
  4 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/man/xinput.man b/man/xinput.man
index 085a5fc..f70fe8c 100644
--- a/man/xinput.man
+++ b/man/xinput.man
@@ -140,9 +140,9 @@ Set the ClientPointer for the client owning \fIwindow\fP to 
\fImaster\fP.
  \fImaster\fP must specify a master pointer.
  .PP
  .TP 8
-.B --map-to-crtc \fIdevice\fP \fIcrtc\fP

Are any scripts using this old name?

+.B --map-to-output \fIdevice\fP \fIcrtc\fP
  Restricts the movements of the absolute \fIdevice\fP to the RandR
-\fIcrtc\fP. The CRTC name must match a currently connected CRTC (see
+\fIcrtc\fP. The output name must match a currently connected output (see

It might be worth mentioning exactly which crtc this is referring to (i.e., the one the given output is attached to).

Also, I don't think it's enough for the output to be connected. It has to actually be bound to an active crtc, right?

  \fIxrandr(__appmansuffix__)\fP). If the NVIDIA binary driver is
  detected or RandR 1.2 or later is not available, a Xinerama output may be


  specified as "HEAD-N", with N being the Xinerama screen number. This option
diff --git a/src/transform.c b/src/transform.c
index c1a065f..cffc3af 100644
--- a/src/transform.c
+++ b/src/transform.c
@@ -136,7 +136,7 @@ set_transformation_matrix(Display *dpy, Matrix *m, int 
offset_x, int offset_y,
  }

  static int
-map_crtc_xrandr(Display *dpy, int deviceid, const char *output_name)
+map_output_xrandr(Display *dpy, int deviceid, const char *output_name)
  {
      int i, found = 0;
      int rc = EXIT_FAILURE;

Not really related to this change, but this function leaks one or more XRRCrtcInfo structures (they're allocated by XRRGetCrtcInfo, to be freed by the caller).

@@ -178,7 +178,7 @@ map_crtc_xrandr(Display *dpy, int deviceid, const char 
*output_name)
  }

  static int
-map_crtc_xinerama(Display *dpy, int deviceid, const char *output_name)
+map_output_xinerama(Display *dpy, int deviceid, const char *output_name)
  {
      const char *prefix = "HEAD-";
      XineramaScreenInfo *screens = NULL;
@@ -229,11 +229,11 @@ out:
  }

  int
-map_to_crtc(Display *dpy, int argc, char *argv[], char *name, char *desc)
+map_to_output(Display *dpy, int argc, char *argv[], char *name, char *desc)
  {
      int opcode, event, error;
      int maj, min;
-    char *crtc_name;
+    char *output_name;
      XIDeviceInfo *info;

      if (argc<  2)
@@ -249,15 +249,15 @@ map_to_crtc(Display *dpy, int argc, char *argv[], char 
*name, char *desc)
          return EXIT_FAILURE;
      }

-    crtc_name = argv[1];
+    output_name = argv[1];

      /* Check for RandR 1.2. Server bug causes the NVIDIA driver to
-     * report with RandR 1.3 support but it doesn't expose RandR CRTCs.
+     * report with RandR 1.3 support but it doesn't expose RandR outputs.
       * Force Xinerama if NV-CONTROL is present */
      if (XQueryExtension(dpy, "NV-CONTROL",&opcode,&event,&error) ||

Eww... this will disallow using RandR outputs on X screens driven by other drivers when the NVIDIA driver is driving any screen. Could you check for the server-generated output named "default" instead?

-- Aaron

          !XQueryExtension(dpy, "RANDR",&opcode,&event,&error) ||
          !XRRQueryVersion(dpy,&maj,&min) || (maj * 1000 + min)<  1002)
-       return map_crtc_xinerama(dpy, info->deviceid, crtc_name);
+       return map_output_xinerama(dpy, info->deviceid, output_name);
      else
-       return map_crtc_xrandr(dpy, info->deviceid, crtc_name);
+       return map_output_xrandr(dpy, info->deviceid, output_name);
  }
diff --git a/src/xinput.c b/src/xinput.c
index 80a1789..66b967b 100644
--- a/src/xinput.c
+++ b/src/xinput.c
@@ -104,9 +104,9 @@ static entry drivers[] =
        "<device>",
        test_xi2,
      },
-    { "map-to-crtc",
-      "<device>  <crtc name>",
-      map_to_crtc,
+    { "map-to-output",
+      "<device>  <output name>",
+      map_to_output,
      },
  #endif
      { "list-props",
diff --git a/src/xinput.h b/src/xinput.h
index 94b8f3c..b420e61 100644
--- a/src/xinput.h
+++ b/src/xinput.h
@@ -77,6 +77,6 @@ int change_attachment( Display* display, int argc, char 
*argv[], char *prog_name
  int float_device( Display* display, int argc, char *argv[], char *prog_name, 
char *prog_desc);
  int set_clientpointer( Display* display, int argc, char *argv[], char 
*prog_name, char *prog_desc);
  int test_xi2( Display* display, int argc, char *argv[], char *prog_name, char 
*prog_desc);
-int map_to_crtc( Display* display, int argc, char *argv[], char *prog_name, 
char *prog_desc);
+int map_to_output( Display* display, int argc, char *argv[], char *prog_name, 
char *prog_desc);

  /* end of xinput.h */

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to