- Notifications
You must be signed in to change notification settings - Fork4.9k
Commit5eb73fc
committed
Reformat imath.c macro to remove -Wmisleading-indentation warnings.
Recent versions of gcc whine about the admittedly-completely-illegibleformatting of this macro. We've not noticed for a few reasons:* In v12 and up, the problem is gone thanks to48e24ba.(Back-patching that doesn't seem prudent, though, so this patchjust manually improves the macro's formatting.)* Buildfarm animals that might have complained, such as caiman,do not because they use --with-openssl and so don't build imath.c.* In a manual run such as "make all check-world", you won't see thewarning because it gets buried in an install.log file. You have todo "make -C contrib all" or the like to see it.I noticed this because in older branches, the last bit doesn'thappen so "check-world" actually does spew the warnings to stderr.Maybe we should rethink how that works, because the newer behavioris not an improvement IMO.Back-patch down to 9.2, pursuant to newly-established project policyabout keeping out-of-support branches buildable.Discussion:https://postgr.es/m/d0316012-ece7-7b7e-2d36-9c38cb77cb3b@enterprisedb.com1 parent3fff9b6 commit5eb73fc
1 file changed
+7
-2
lines changedLines changed: 7 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
129 | 129 |
| |
130 | 130 |
| |
131 | 131 |
| |
132 |
| - | |
133 |
| - | |
| 132 | + | |
| 133 | + | |
| 134 | + | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
134 | 139 |
| |
135 | 140 |
| |
136 | 141 |
| |
|
0 commit comments
Comments
(0)