- Notifications
You must be signed in to change notification settings - Fork5
Commit820bdcc
committed
Remove a useless PG_GETARG_DATUM() call from jsonb_build_array.
This loop uselessly fetched the argument after the one it's currentlylooking at. No real harm is done since we couldn't possibly fetch offthe end of memory, but it's confusing to the reader.Also remove a duplicate (and therefore confusing) PG_ARGISNULL check injsonb_build_object.I happened to notice these things while trolling for missed null-argchecks earlier today. Back-patch to 9.5, not because there is anyreal bug, but just because 9.5 and HEAD are still in sync in thisfile and we might as well keep them so.In passing, re-pgindent.1 parent3ef16c4 commit820bdcc
1 file changed
+5
-11
lines changedLines changed: 5 additions & 11 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
721 | 721 |
| |
722 | 722 |
| |
723 | 723 |
| |
724 |
| - | |
| 724 | + | |
725 | 725 |
| |
726 | 726 |
| |
727 | 727 |
| |
| |||
1187 | 1187 |
| |
1188 | 1188 |
| |
1189 | 1189 |
| |
1190 |
| - | |
1191 | 1190 |
| |
1192 | 1191 |
| |
1193 | 1192 |
| |
| |||
1203 | 1202 |
| |
1204 | 1203 |
| |
1205 | 1204 |
| |
1206 |
| - | |
1207 |
| - | |
1208 |
| - | |
1209 |
| - | |
| 1205 | + | |
1210 | 1206 |
| |
1211 | 1207 |
| |
1212 | 1208 |
| |
| |||
1215 | 1211 |
| |
1216 | 1212 |
| |
1217 | 1213 |
| |
1218 |
| - | |
| 1214 | + | |
1219 | 1215 |
| |
1220 | 1216 |
| |
1221 | 1217 |
| |
| |||
1238 | 1234 |
| |
1239 | 1235 |
| |
1240 | 1236 |
| |
1241 |
| - | |
| 1237 | + | |
1242 | 1238 |
| |
1243 |
| - | |
1244 | 1239 |
| |
1245 | 1240 |
| |
1246 | 1241 |
| |
| |||
1283 | 1278 |
| |
1284 | 1279 |
| |
1285 | 1280 |
| |
1286 |
| - | |
1287 | 1281 |
| |
1288 | 1282 |
| |
1289 | 1283 |
| |
| |||
1300 | 1294 |
| |
1301 | 1295 |
| |
1302 | 1296 |
| |
1303 |
| - | |
| 1297 | + | |
1304 | 1298 |
| |
1305 | 1299 |
| |
1306 | 1300 |
| |
|
0 commit comments
Comments
(0)