- Notifications
You must be signed in to change notification settings - Fork5
Commita40a5d9
committed
Remove extra copying of TupleDescs for heap_create_with_catalog
Some callers were creating copies of tuple descriptors to pass to thatfunction, stating in code comments that it was necessary because itmodified the passed descriptor. Code inspection reveals this not to betrue, and indeed not all callers are passing copies in the first place.So remove the extra ones and the misleading comments about this behavioras well.1 parentbddc35a commita40a5d9
2 files changed
+4
-13
lines changedLines changed: 3 additions & 6 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
591 | 591 |
| |
592 | 592 |
| |
593 | 593 |
| |
594 |
| - | |
595 |
| - | |
| 594 | + | |
596 | 595 |
| |
597 | 596 |
| |
598 | 597 |
| |
| |||
605 | 604 |
| |
606 | 605 |
| |
607 | 606 |
| |
608 |
| - | |
609 |
| - | |
| 607 | + | |
610 | 608 |
| |
611 | 609 |
| |
612 | 610 |
| |
613 | 611 |
| |
614 |
| - | |
615 | 612 |
| |
616 | 613 |
| |
617 | 614 |
| |
| |||
645 | 642 |
| |
646 | 643 |
| |
647 | 644 |
| |
648 |
| - | |
| 645 | + | |
649 | 646 |
| |
650 | 647 |
| |
651 | 648 |
| |
|
Lines changed: 1 addition & 7 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
2394 | 2394 |
| |
2395 | 2395 |
| |
2396 | 2396 |
| |
2397 |
| - | |
2398 | 2397 |
| |
2399 | 2398 |
| |
2400 | 2399 |
| |
| |||
2467 | 2466 |
| |
2468 | 2467 |
| |
2469 | 2468 |
| |
2470 |
| - | |
2471 |
| - | |
2472 |
| - | |
2473 | 2469 |
| |
2474 | 2470 |
| |
2475 | 2471 |
| |
| |||
2478 | 2474 |
| |
2479 | 2475 |
| |
2480 | 2476 |
| |
2481 |
| - | |
| 2477 | + | |
2482 | 2478 |
| |
2483 | 2479 |
| |
2484 | 2480 |
| |
| |||
2492 | 2488 |
| |
2493 | 2489 |
| |
2494 | 2490 |
| |
2495 |
| - | |
2496 |
| - | |
2497 | 2491 |
| |
2498 | 2492 |
| |
2499 | 2493 |
| |
|
0 commit comments
Comments
(0)