- Notifications
You must be signed in to change notification settings - Fork5
Commitce47112
committed
Adjust datatype of ReplicationState.acquired_by.
It was declared as "pid_t", which would be fine except that none ofthe places that printed it in error messages took any thought for thepossibility that it's not equivalent to "int". This leads to warningson some buildfarm members, and could possibly lead to actually wrongerror messages on those platforms. There doesn't seem to be any verygood reason not to just make it "int"; it's only ever assigned fromMyProcPid, which is int. If we want to cope with PIDs that are widerthan int, this is not the place to start.Also, fix the comment, which seems to perhaps be a leftover from a timewhen the field was only a bool?Per buildfarm. Back-patch to 9.5 which has same issue.1 parentfb0f639 commitce47112
1 file changed
+2
-2
lines changedLines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
120 | 120 | | |
121 | 121 | | |
122 | 122 | | |
123 | | - | |
| 123 | + | |
124 | 124 | | |
125 | | - | |
| 125 | + | |
126 | 126 | | |
127 | 127 | | |
128 | 128 | | |
| |||
0 commit comments
Comments
(0)