diff options
author | Jamey Sharp <jamey@minilop.net> | 2010-05-06 21:39:43 -0700 |
---|---|---|
committer | Jamey Sharp <jamey@minilop.net> | 2010-05-12 18:09:55 -0700 |
commit | 65cedf39054cf3a1e695e84ac228cce9e8d48097 (patch) | |
tree | 9a0b41cf70562a6e9e06263486f2fcf1552da46a /mi/miwideline.h | |
parent | afcbbd6dfea51c5b0adca0d720edc02ba0c2dc16 (diff) | |
download | xserver-65cedf39054cf3a1e695e84ac228cce9e8d48097.tar.gz |
Kill DoChangeGC in favor of dixChangeGC.
This doesn't change any behavior, but it isn't clear whether NullClient
is correct in all cases. As ajax says,
> For most of these changes, I think it's correct to use NullClient,
> since they are server-initiated changes and should not fail for (eg)
> xace reasons. ... At any rate, you're certainly not changing any
> semantics by leaving them all as NullClient, so this patch can't be
> more wrong than before.
Signed-off-by: Jamey Sharp <jamey@minilop.net>
Reviewed-by: Keith Packard <keithp@keithp.com>
Diffstat (limited to 'mi/miwideline.h')
-rw-r--r-- | mi/miwideline.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/mi/miwideline.h b/mi/miwideline.h index 1665ebfb1..2ff36ed6b 100644 --- a/mi/miwideline.h +++ b/mi/miwideline.h @@ -78,13 +78,13 @@ typedef struct _LineFace { #define MILINESETPIXEL(pDrawable, pGC, pixel, oldPixel) { \ oldPixel = pGC->fgPixel; \ if (pixel != oldPixel) { \ - DoChangeGC (pGC, GCForeground, (XID *) &pixel, FALSE); \ + dixChangeGC (NullClient, pGC, GCForeground, (XID *) &pixel, NULL); \ ValidateGC (pDrawable, pGC); \ } \ } #define MILINERESETPIXEL(pDrawable, pGC, pixel, oldPixel) { \ if (pixel != oldPixel) { \ - DoChangeGC (pGC, GCForeground, (XID *) &oldPixel, FALSE); \ + dixChangeGC (NullClient, pGC, GCForeground, (XID *) &oldPixel, NULL); \ ValidateGC (pDrawable, pGC); \ } \ } |