tag name | btree-private-unions_2020-03-11 (9ac799018322cf39c383b89848da7e3de326bd26) |
tag date | 2020-03-11 20:37:08 -0700 |
tagged by | Darrick J. Wong <darrick.wong@oracle.com> |
tagged object | commit 30d91fbcef... |
xfs: make btree cursor private unions anonymous
Dave injured his finger and is having difficulty typing, so I fixed a few
things that I didn't like about the v1 patchset and am sending it back out
ahead of resending the btree bulk loading patchset.
--D
This is a "make things less verbose" cleanup from looking at the
changes Darrick is making to add a staging/fake cursor to the union
for bulk btree loading.
The process is to create a @defines of the new name to the existing
union name, then replace all users of each union via a script. Then
the union is made anonymous and the members renamed to match the new
code. Then the #defines get removed.
We do this for the bc_private union, then we name the ag and btree
structures and make them use anonymous unions internally via the
same process.
That means we go from doubly nested private stuff like this:
cur->bc_private.a.priv.abt.active
To the much cleaner, less verbose and more readable:
cur->bc_ag.abt.active
Simples, yes?
[The original] series can be found at:
https://git.kernel.org/pub/scm/linux/kernel/git/dgc/linux-xfs.git/h?xfs-btree-cursor-cleanup
Note: the code changes are all scripted, I have not done any
followup to do things like aggregate split lines back into single
lines as that is out of scope of the structure definition cleanup
I'm trying to acheive here. That can be done in future as we modify
the code that now has lines that can be merged....
Signed-off-by: Dave Chinner <dchinner@redhat.com>
-----BEGIN PGP SIGNATURE-----
iQIzBAABCgAdFiEEUzaAxoMeQq6m2jMV+H93GTRKtOsFAl5prmQACgkQ+H93GTRK
tOvmCw/+KDhA9CLYN2nXZy7H6Dt9myqwaUO83L9ksX8j8eP1e11c86FyZCYiiLfJ
I54lr4I2aIcaf6inL/OqnHw+uOxIja1wFvXCnLgnqbvW+gWx2UPWo2+wUDGRAvDb
K51IuGeioExHpctKvjtnsnjemkXIVyqRYA/MxNku1nKmBcMJ2yEg2X5PFjhoRpsn
td5c7BvnfiFJe+tZs2YHbwVbpy6Tn2WpFYTj2OdDtEWGT+hU0QiCUNu9ep6J+9nh
65hmppgPYiCwPdM5XyuT2ZCIXxiDzuLUEilQJ6QM1MsfuyYcIcjahOcW1EF5wdlz
YtAkf+mVLYBtWNDTKR7aSgH8xuLPeB6p1OzGIP4pmt3vfxZcAgMBq47MEbMFlUOS
adUCqP/t6kX4+kc8ft7IAdJNRMtesJIm6BTvoA9BYex320vGwyAfDGaH9iy2Dfo5
zMHEoob3LgEvGI20A+79UazzTKmOOQP5pzzQ6nz1t43omUbfyOlw44rezeiC2Vtt
Fv7gDXpe1M/PITmaibgbeJ2tRe6segkaP+vxaCM1GMmGtDl+dTxlIA5gv5+fimBW
lGV3oij7mK/XCavToVYt3ShgZQBUedlp3DghHmntuK2J7pt2xBf9sBfTPzYvQiFV
cAe3Y28o54BPrroylzjLPZHaY75cyGiV8GgntgCEp2ghYjE9cTo=
=2PZK
-----END PGP SIGNATURE-----