forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitb9d232b
committed
Use "ssize_t" not "long" in max_stack_depth-related code.
This change adapts these functions to the machine's address widthwithout depending on "long" to be the right size. (It isn't onWin64, for example.) While it seems unlikely anyone would careto run with a stack depth limit exceeding 2GB, this is part of ageneral push to avoid using type "long" to represent memory sizes.It's convenient to use ssize_t rather than the perhaps-more-obviouschoice of size_t/Size, because the code involved depends on workingwith a signed data type. Our MAX_KILOBYTES limit already ensuresthat ssize_t will be sufficient to represent the maximum value ofmax_stack_depth.Extracted from a larger patch by Vladlen, plus additional hackeryby me.Author: Vladlen Popolitov <v.popolitov@postgrespro.ru>Author: Tom Lane <tgl@sss.pgh.pa.us>Discussion:https://postgr.es/m/1a01f0-66ec2d80-3b-68487680@275952171 parentb9aa416 commitb9d232b
File tree
3 files changed
+22
-18
lines changed- src
- backend/utils/misc
- include
3 files changed
+22
-18
lines changedLines changed: 3 additions & 3 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1589 | 1589 |
| |
1590 | 1590 |
| |
1591 | 1591 |
| |
1592 |
| - | |
| 1592 | + | |
1593 | 1593 |
| |
1594 | 1594 |
| |
1595 | 1595 |
| |
| |||
1613 | 1613 |
| |
1614 | 1614 |
| |
1615 | 1615 |
| |
1616 |
| - | |
| 1616 | + | |
1617 | 1617 |
| |
1618 | 1618 |
| |
1619 | 1619 |
| |
| |||
1627 | 1627 |
| |
1628 | 1628 |
| |
1629 | 1629 |
| |
1630 |
| - | |
| 1630 | + | |
1631 | 1631 |
| |
1632 | 1632 |
| |
1633 | 1633 |
| |
|
Lines changed: 17 additions & 13 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
26 | 26 |
| |
27 | 27 |
| |
28 | 28 |
| |
29 |
| - | |
| 29 | + | |
30 | 30 |
| |
31 | 31 |
| |
32 | 32 |
| |
| |||
109 | 109 |
| |
110 | 110 |
| |
111 | 111 |
| |
112 |
| - | |
| 112 | + | |
113 | 113 |
| |
114 | 114 |
| |
115 | 115 |
| |
116 | 116 |
| |
117 |
| - | |
| 117 | + | |
118 | 118 |
| |
119 | 119 |
| |
120 | 120 |
| |
| |||
141 | 141 |
| |
142 | 142 |
| |
143 | 143 |
| |
144 |
| - | |
145 |
| - | |
| 144 | + | |
| 145 | + | |
146 | 146 |
| |
147 | 147 |
| |
148 | 148 |
| |
149 |
| - | |
150 |
| - | |
| 149 | + | |
| 150 | + | |
151 | 151 |
| |
152 | 152 |
| |
153 | 153 |
| |
| |||
158 | 158 |
| |
159 | 159 |
| |
160 | 160 |
| |
161 |
| - | |
| 161 | + | |
162 | 162 |
| |
163 | 163 |
| |
164 | 164 |
| |
| |||
167 | 167 |
| |
168 | 168 |
| |
169 | 169 |
| |
| 170 | + | |
| 171 | + | |
| 172 | + | |
| 173 | + | |
170 | 174 |
| |
171 |
| - | |
| 175 | + | |
172 | 176 |
| |
173 | 177 |
| |
174 | 178 |
| |
175 |
| - | |
| 179 | + | |
176 | 180 |
| |
177 | 181 |
| |
178 | 182 |
| |
| |||
182 | 186 |
| |
183 | 187 |
| |
184 | 188 |
| |
185 |
| - | |
| 189 | + | |
186 | 190 |
| |
187 |
| - | |
188 |
| - | |
| 191 | + | |
| 192 | + | |
189 | 193 |
| |
190 | 194 |
| |
191 | 195 |
| |
|
Lines changed: 2 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
293 | 293 |
| |
294 | 294 |
| |
295 | 295 |
| |
296 |
| - | |
| 296 | + | |
297 | 297 |
| |
298 | 298 |
| |
299 | 299 |
| |
300 | 300 |
| |
301 | 301 |
| |
302 | 302 |
| |
303 | 303 |
| |
304 |
| - | |
| 304 | + | |
305 | 305 |
| |
306 | 306 |
| |
307 | 307 |
| |
|
0 commit comments
Comments
(0)