forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit70353e4
committed
Sync typedefs.list with buildfarm, for real this time.
In commitda256a4, I manually added some typedef names to thebuildfarm-generated list so as not to cause any formatting regressionscompared to the prior manually-updated list.About half of the additions were injection-point-related names.It turns out that those were missing because none of the buildfarmanimals contributing typedef lists were building with--enable-injection-points. I rectified that on my animal sifaka,and now those are in the list available from the buildfarm.The other half were typedefs that didn't show up in the generated listbecause our method for collecting that doesn't catch names that arenot used to declare any C variables or fields. Such a typedef namedoesn't really add a lot of value, so we decided to get rid of them,and that's now been done in commits110eb4a andbe5942a. (Note:I'm pretty sure there are some remaining cases of that, but we'vealready accepted the ensuing odd formatting of the typedef declarationitself. The present fixes only dealt with typedefs that had beenmanually added to typedefs.list during the v17 development cycle.)Hence, we can now install a verbatim copy of the buildfarm's listand not have it affect anything. The only change is to addInjectionPointCallback, which I'd omitted fromda256a4 becauseit chanced not to affect any pgindent decisions.Discussion:https://postgr.es/m/1919000.1715815925@sss.pgh.pa.us1 parente9b7aee commit70353e4
1 file changed
+1
-0
lines changedLines changed: 1 addition & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1231 | 1231 | | |
1232 | 1232 | | |
1233 | 1233 | | |
| 1234 | + | |
1234 | 1235 | | |
1235 | 1236 | | |
1236 | 1237 | | |
| |||
0 commit comments
Comments
(0)