Make listed lengths match data shown & values in header for
RRSetProviderOffloadSink & RRSetProviderOutputSource
Signed-off-by: Alan Coopersmith
---
randrproto.txt |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/randrproto.txt b/randrproto.txt
index db7088d..2bf065f 10
Uses values found in randr.h
Signed-off-by: Alan Coopersmith
---
randrproto.txt |7 +++
1 file changed, 7 insertions(+)
diff --git a/randrproto.txt b/randrproto.txt
index b52a7c5..9887ded 100644
--- a/randrproto.txt
+++ b/randrproto.txt
@@ -2169,6 +2169,13 @@ A.1 Common Types
2
Present in header & encoding section, just not in the descriptions of
RRSetProviderOffloadSink & RRSetProviderOutputSource
Signed-off-by: Alan Coopersmith
---
randrproto.txt |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/randrproto.txt b/randrproto.txt
index 9887ded..
The encoding is missing the 8 bytes of padding at the end of the
base reply struct before the variable length lists begin.
Signed-off-by: Alan Coopersmith
---
randrproto.txt |1 +
1 file changed, 1 insertion(+)
diff --git a/randrproto.txt b/randrproto.txt
index 91b7d52..b52a7c5 100644
--- a
These values match the RRNotify_* definitions in randrproto.h and
don't overlap with the RROutput*Notify codes from RandR 1.2.
Signed-off-by: Alan Coopersmith
---
randrproto.txt |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/randrproto.txt b/randrproto.txt
index e0b
The padding for RRGetCrtcGamma & RRSetCrtcGamma is a variable p,
not a fixed 2 bytes, since it depends on how many values are
included in the lists.
Signed-off-by: Alan Coopersmith
---
randrproto.txt |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/randrproto.txt b/rand
The description section matches randrproto.h - neither of which have
the "maximum masters" or "flags" members, but do have padding before
the providers list starts.
Signed-off-by: Alan Coopersmith
---
randrproto.txt |5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/rand
The encoding section matches randrproto.h - neither of which have the
config-timestamp, root, or crtc members.
Signed-off-by: Alan Coopersmith
---
randrproto.txt | 11 +++
1 file changed, 3 insertions(+), 8 deletions(-)
diff --git a/randrproto.txt b/randrproto.txt
index a6fe431..29cba
Each OUTPUT is 4 bytes, not 8.
Signed-off-by: Alan Coopersmith
---
randrproto.txt |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/randrproto.txt b/randrproto.txt
index 478c507..27fc2f3 100644
--- a/randrproto.txt
+++ b/randrproto.txt
@@ -2527,7 +2527,7 @@ A.2.1 Protoco
At least I'm assuming "providre" is a typo, not just a non-American variant.
Signed-off-by: Alan Coopersmith
---
randrproto.txt | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/randrproto.txt b/randrproto.txt
index e263108..8c6292a 100644
--- a/randrproto.tx
Make definitions more consistent about use of {} for enums, [] for structs
Reflect the incorporation of more types from the Render spec in 1.3
Signed-off-by: Alan Coopersmith
---
randrproto.txt | 47 +--
1 file changed, 33 insertions(+), 14 deletions
One side benefit of adding RANDR support to xscope is it's basically
writing a second independent implementation of the protocol code from
the spec, since it doesn't use the Xlib, XCB, or Xserver code for decoding.
Thus, inconsistencies and nits in the spec were uncovered, and I've tried
to guess
(1 + 1 + 2 + 4 + 32) / 4 is 10 words, not 12
10 also matches the sz_xRRCreateModeReq value of 40 in randrproto.h
Signed-off-by: Alan Coopersmith
---
randrproto.txt |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/randrproto.txt b/randrproto.txt
index 7ff573b..e263108 100644
If you run "xscope -r -v0 > xscope.raw", then you can later run
"xscope -f xscope.raw" to decode the data.
Mainly adding this to aid in testing decoding of new extensions,
so I don't have to re-run the commands over and over, just replay
a log file. (Which may or may not have been edited to incl
All versions now perform same check for XtransConn data to decide
whether to use _X11TransClose or regular close, which is needed for
being able to read data from pre-recorded files instead of live sockets.
Signed-off-by: Alan Coopersmith
---
fd.c| 23 +--
fd.h|
All errors currently recognized by xscope fall into two forms,
with or without a 32-bit value to print as a bad value, so use
common implementations for those two forms so we can stop
duplicating that code for every new extension-defined error.
Signed-off-by: Alan Coopersmith
---
print11.c
Avoids having to try to time a ^C after the signal handler is set up
but before any data is handled.
Signed-off-by: Alan Coopersmith
---
man/xscope.man |9 +++--
scope.c|5 +
2 files changed, 12 insertions(+), 2 deletions(-)
diff --git a/man/xscope.man b/man/xscope.man
i
Currently used for core protocol window properties, but will be
used for extensions that have similar property handling requests
in the future.
Signed-off-by: Alan Coopersmith
---
print11.c | 16 +---
proto.h |3 +++
prtype.c | 18 ++
3 files changed, 26 in
Prints atoms, cardinals, integers, and windows as formatted values
instead of lists of bytes.
Signed-off-by: Alan Coopersmith
---
prtype.c | 45 +++--
1 file changed, 43 insertions(+), 2 deletions(-)
diff --git a/prtype.c b/prtype.c
index 04f36cf..182e8
We don't print any other data from packet when Verbose == 0, just had
a few random "data: (6)" type lines sneak through.
Signed-off-by: Alan Coopersmith
---
print11.c |2 ++
prtype.c |3 +++
2 files changed, 5 insertions(+)
diff --git a/print11.c b/print11.c
index b2b61fb..7935b84 1006
Signed-off-by: Alan Coopersmith
---
configure.ac |2 +-
prtype.c | 19 ---
scope.c | 24 +++-
scope.h |3 ++-
4 files changed, 42 insertions(+), 6 deletions(-)
diff --git a/configure.ac b/configure.ac
index 74d0d53..dee7b66 100644
---
Signed-off-by: Alan Coopersmith
---
server.c |8
x11.h| 10 ++
2 files changed, 10 insertions(+), 8 deletions(-)
diff --git a/server.c b/server.c
index 3369460..5226a03 100644
--- a/server.c
+++ b/server.c
@@ -138,7 +138,7 @@ pad(long n)
return ((n + 3) & ~0x3);
Uses them to display strings instead of just numeric ids for atoms
beyond the builtin set in other requests, such as property lookups.
Signed-off-by: Alan Coopersmith
---
COPYING |3 +-
print11.c | 92 -
proto.h |2 ++
prt
Used for GC struct members, which are defined in the protocol as 32-bit,
so no need to waste time and memory copying into 64-bit longs everywhere
Signed-off-by: Alan Coopersmith
---
print11.c |4 ++--
prtype.c |2 +-
table11.c | 16
x11.h | 16
Adds a getreqlen inline function to handle the big requests check.
As noted in the comment, it must be called *before* the printreqlen
macro does its own check, since printreqlen "fixes" the buf pointer
when it's done to align all the remaining fields with their correct
positions after skipping ove
Signed-off-by: Alan Coopersmith
---
common.c |2 +-
fd.c |8 +---
prtype.c |4 ++--
scope.c |6 +++---
4 files changed, 11 insertions(+), 9 deletions(-)
diff --git a/common.c b/common.c
index 3c59c0b..0492e47 100644
--- a/common.c
+++ b/common.c
@@ -217,7 +217,7 @@ SetUp
instead of shifting the whole word as each byte is loaded into place
Signed-off-by: Alan Coopersmith
---
x11.h |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/x11.h b/x11.h
index ed0f3d6..a3c8cb9 100644
--- a/x11.h
+++ b/x11.h
@@ -547,8 +547,8 @@ ILong(const unsigned c
Signed-off-by: Alan Coopersmith
---
scope.h |3 ++-
server.c | 57
x11.h| 63 +++---
3 files changed, 58 insertions(+), 65 deletions(-)
diff --git a/scope.h b/scope.h
ind
Clears gcc warning from casting a const char * to char *:
server.c: In function `SaveBytes':
server.c:203: warning: cast discards qualifiers from pointer target type
Signed-off-by: Alan Coopersmith
---
server.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/server.c
Signed-off-by: Alan Coopersmith
---
common.c |2 +-
fd.c |2 +-
server.c |6 +++---
3 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/common.c b/common.c
index 8c88aae..3997464 100644
--- a/common.c
+++ b/common.c
@@ -277,7 +277,7 @@ SetUpConnectionSocket(int iport, v
Signed-off-by: Alan Coopersmith
---
common.c |4 ++--
scope.c |4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/common.c b/common.c
index 0492e47..8c88aae 100644
--- a/common.c
+++ b/common.c
@@ -107,14 +107,14 @@ SignalINT(int sig)
Interrupt = 1;
}
-static
Signed-off-by: Alan Coopersmith
---
scope.c | 25 +
1 file changed, 13 insertions(+), 12 deletions(-)
diff --git a/scope.c b/scope.c
index 9928bad..2f9befc 100644
--- a/scope.c
+++ b/scope.c
@@ -649,18 +649,19 @@ GetScopePort(void)
static void
Usage(void)
{
-fpri
Sure I thought, it will be easy to update the RANDR support in xscope to the
new version 1.4 - it's almost there, right?
Nope, turns out xscope's RANDR support was still 0.x only - no 1.0 or later.
And around every corner trying to get to 1.4 was something else to fix first.
24 commits later and
Signed-off-by: Alan Coopersmith
---
common.c |3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/common.c b/common.c
index d6f4d7c..c0e9fd6 100644
--- a/common.c
+++ b/common.c
@@ -187,8 +187,6 @@ static int ListenTransCount;
#include /* struct sockaddr_in */
#incl
Fixes compiler warning:
"fd.c", line 481: warning: statement not reached
Signed-off-by: Alan Coopersmith
---
fd.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fd.c b/fd.c
index 00a68a2..8b85460 100644
--- a/fd.c
+++ b/fd.c
@@ -477,8 +477,8 @@ MainLoop(void)
Signed-off-by: Alan Coopersmith
---
common.c|3 +--
decode11.c | 14 ++
decode_glx.c|3 +--
decode_render.c |8 +++-
fd.c|7 ++-
print_render.c |3 +--
prtype.c| 49 -
Signed-off-by: Alan Coopersmith
---
print11.c |8
x11.h |2 +-
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/print11.c b/print11.c
index e7da143..36b335f 100644
--- a/print11.c
+++ b/print11.c
@@ -179,10 +179,10 @@ PrintSuccessfulSetUpReply(const unsigned cha
Fixes compiler warnings when building app/xrandr:
xrandr.c: In function ‘crtc_set_transform’:
xrandr.c:1459:9: warning: passing argument 4 of ‘XRRSetCrtcTransform’ discards
qualifiers from pointer target type
X11/extensions/Xrandr.h:419:1: note: expected ‘char *’ but argument is of type
‘const c
On 08/29/2012 09:12 PM, Peter Hutterer wrote:
These messages are not correct when the server isn't started with the dummy
driver.
Signed-off-by: Peter Hutterer
---
README | 7 +++
configure.ac| 2 +-
src/xorg-gtest_main.cpp | 4 ++--
src/xserver.cpp
On 08/31/2012 09:13 AM, Markus Trippelsdorf wrote:
(EE) BUG: triggered 'if (inSignalContext)'
(EE) BUG:
/var/tmp/portage/x11-base/xorg-server-1.12.99.905/work/xorg-server-1.12.99.905/os/log.c:472
in LogVMessageVerb()
(EE) Warning: attempting to log data in a signal unsafe manner while in signal
(EE) BUG: triggered 'if (inSignalContext)'
(EE) BUG:
/var/tmp/portage/x11-base/xorg-server-1.12.99.905/work/xorg-server-1.12.99.905/os/log.c:472
in LogVMessageVerb()
(EE) Warning: attempting to log data in a signal unsafe manner while in signal
context. Please update to check inSignalContext and
On 31 August 2012 11:49, Simon Schubert <2...@0x2c.org> wrote:
> On 08/31/2012 09:18 AM, Michal Suchanek wrote:
>> On 31 August 2012 08:24, Jeremy Huddleston Sequoia
>> wrote:
>>> Simon,
>>>
>>> Do you think you'll be able to come up with a fix for this regression
>>> tomorrow (ie before Saturday
On 08/31/2012 09:18 AM, Michal Suchanek wrote:
> On 31 August 2012 08:24, Jeremy Huddleston Sequoia
> wrote:
>> Simon,
>>
>> Do you think you'll be able to come up with a fix for this regression
>> tomorrow (ie before Saturday)? If not, we should revert the change from
>> master before we tag 1
On 31 August 2012 08:24, Jeremy Huddleston Sequoia
wrote:
> Simon,
>
> Do you think you'll be able to come up with a fix for this regression
> tomorrow (ie before Saturday)? If not, we should revert the change from
> master before we tag 1.13 on Tuesday. I'll also revert it from
> server-1.12
44 matches
Mail list logo