|
|
98518b3 |
From f1070c01d616c5f21f939d5ebc533738779451ac Mon Sep 17 00:00:00 2001
|
|
|
98518b3 |
From: Peter Hutterer <peter.hutterer@who-t.net>
|
|
|
98518b3 |
Date: Tue, 5 Jul 2022 12:40:47 +1000
|
|
|
98518b3 |
Subject: [PATCH xserver 1/3] xkb: switch to array index loops to moving
|
|
|
98518b3 |
pointers
|
|
|
98518b3 |
|
|
|
98518b3 |
Most similar loops here use a pointer that advances with each loop
|
|
|
98518b3 |
iteration, let's do the same here for consistency.
|
|
|
98518b3 |
|
|
|
98518b3 |
No functional changes.
|
|
|
98518b3 |
|
|
|
98518b3 |
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
|
|
|
98518b3 |
Reviewed-by: Olivier Fourdan <ofourdan@redhat.com>
|
|
|
98518b3 |
---
|
|
|
98518b3 |
xkb/xkb.c | 20 ++++++++++----------
|
|
|
98518b3 |
1 file changed, 10 insertions(+), 10 deletions(-)
|
|
|
98518b3 |
|
|
|
98518b3 |
diff --git a/xkb/xkb.c b/xkb/xkb.c
|
|
|
98518b3 |
index a29262c24..64e52611e 100644
|
|
|
98518b3 |
--- a/xkb/xkb.c
|
|
|
98518b3 |
+++ b/xkb/xkb.c
|
|
|
98518b3 |
@@ -5368,16 +5368,16 @@ _CheckSetSections(XkbGeometryPtr geom,
|
|
|
98518b3 |
row->left = rWire->left;
|
|
|
98518b3 |
row->vertical = rWire->vertical;
|
|
|
98518b3 |
kWire = (xkbKeyWireDesc *) &rWire[1];
|
|
|
98518b3 |
- for (k = 0; k < rWire->nKeys; k++) {
|
|
|
98518b3 |
+ for (k = 0; k < rWire->nKeys; k++, kWire++) {
|
|
|
98518b3 |
XkbKeyPtr key;
|
|
|
98518b3 |
|
|
|
98518b3 |
key = XkbAddGeomKey(row);
|
|
|
98518b3 |
if (!key)
|
|
|
98518b3 |
return BadAlloc;
|
|
|
98518b3 |
- memcpy(key->name.name, kWire[k].name, XkbKeyNameLength);
|
|
|
98518b3 |
- key->gap = kWire[k].gap;
|
|
|
98518b3 |
- key->shape_ndx = kWire[k].shapeNdx;
|
|
|
98518b3 |
- key->color_ndx = kWire[k].colorNdx;
|
|
|
98518b3 |
+ memcpy(key->name.name, kWire->name, XkbKeyNameLength);
|
|
|
98518b3 |
+ key->gap = kWire->gap;
|
|
|
98518b3 |
+ key->shape_ndx = kWire->shapeNdx;
|
|
|
98518b3 |
+ key->color_ndx = kWire->colorNdx;
|
|
|
98518b3 |
if (key->shape_ndx >= geom->num_shapes) {
|
|
|
98518b3 |
client->errorValue = _XkbErrCode3(0x10, key->shape_ndx,
|
|
|
98518b3 |
geom->num_shapes);
|
|
|
98518b3 |
@@ -5389,7 +5389,7 @@ _CheckSetSections(XkbGeometryPtr geom,
|
|
|
98518b3 |
return BadMatch;
|
|
|
98518b3 |
}
|
|
|
98518b3 |
}
|
|
|
98518b3 |
- rWire = (xkbRowWireDesc *) &kWire[rWire->nKeys];
|
|
|
98518b3 |
+ rWire = (xkbRowWireDesc *)kWire;
|
|
|
98518b3 |
}
|
|
|
98518b3 |
wire = (char *) rWire;
|
|
|
98518b3 |
if (sWire->nDoodads > 0) {
|
|
|
98518b3 |
@@ -5454,16 +5454,16 @@ _CheckSetShapes(XkbGeometryPtr geom,
|
|
|
98518b3 |
return BadAlloc;
|
|
|
98518b3 |
ol->corner_radius = olWire->cornerRadius;
|
|
|
98518b3 |
ptWire = (xkbPointWireDesc *) &olWire[1];
|
|
|
98518b3 |
- for (p = 0, pt = ol->points; p < olWire->nPoints; p++, pt++) {
|
|
|
98518b3 |
- pt->x = ptWire[p].x;
|
|
|
98518b3 |
- pt->y = ptWire[p].y;
|
|
|
98518b3 |
+ for (p = 0, pt = ol->points; p < olWire->nPoints; p++, pt++, ptWire++) {
|
|
|
98518b3 |
+ pt->x = ptWire->x;
|
|
|
98518b3 |
+ pt->y = ptWire->y;
|
|
|
98518b3 |
if (client->swapped) {
|
|
|
98518b3 |
swaps(&pt->x);
|
|
|
98518b3 |
swaps(&pt->y);
|
|
|
98518b3 |
}
|
|
|
98518b3 |
}
|
|
|
98518b3 |
ol->num_points = olWire->nPoints;
|
|
|
98518b3 |
- olWire = (xkbOutlineWireDesc *) (&ptWire[olWire->nPoints]);
|
|
|
98518b3 |
+ olWire = (xkbOutlineWireDesc *)ptWire;
|
|
|
98518b3 |
}
|
|
|
98518b3 |
if (shapeWire->primaryNdx != XkbNoShape)
|
|
|
98518b3 |
shape->primary = &shape->outlines[shapeWire->primaryNdx];
|
|
|
98518b3 |
--
|
|
|
98518b3 |
2.36.1
|
|
|
98518b3 |
|