forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit235eb4d
committed
Simplify our Assert infrastructure a little.
Remove the Trap and TrapMacro macros, which were nearly unusedand confusingly had the opposite condition polarity from theotherwise-functionally-equivalent Assert macros.Having done that, it's very hard to justify carrying the errorTypeargument of ExceptionalCondition, so drop that too, and justlet it assume everything's an Assert. This saves about 64Kof code space as of current HEAD.Discussion:https://postgr.es/m/3928703.1665345117@sss.pgh.pa.us1 parent6291b25 commit235eb4d
File tree
5 files changed
+22
-51
lines changed- contrib/amcheck
- src
- backend/utils/error
- include
5 files changed
+22
-51
lines changedLines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1319 | 1319 |
| |
1320 | 1320 |
| |
1321 | 1321 |
| |
1322 |
| - | |
| 1322 | + | |
1323 | 1323 |
| |
1324 | 1324 |
| |
1325 | 1325 |
| |
|
Lines changed: 3 additions & 6 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
28 | 28 |
| |
29 | 29 |
| |
30 | 30 |
| |
31 |
| - | |
32 | 31 |
| |
33 | 32 |
| |
34 | 33 |
| |
35 | 34 |
| |
36 | 35 |
| |
37 |
| - | |
38 |
| - | |
| 36 | + | |
39 | 37 |
| |
40 | 38 |
| |
41 | 39 |
| |
42 |
| - | |
43 |
| - | |
44 |
| - | |
| 40 | + | |
| 41 | + | |
45 | 42 |
| |
46 | 43 |
| |
47 | 44 |
| |
|
Lines changed: 1 addition & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1832 | 1832 |
| |
1833 | 1833 |
| |
1834 | 1834 |
| |
1835 |
| - | |
1836 |
| - | |
| 1835 | + | |
1837 | 1836 |
| |
1838 | 1837 |
| |
1839 | 1838 |
| |
|
Lines changed: 16 additions & 41 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
796 | 796 |
| |
797 | 797 |
| |
798 | 798 |
| |
799 |
| - | |
800 |
| - | |
801 | 799 |
| |
802 | 800 |
| |
803 | 801 |
| |
| |||
811 | 809 |
| |
812 | 810 |
| |
813 | 811 |
| |
814 |
| - | |
815 |
| - | |
| 812 | + | |
| 813 | + | |
816 | 814 |
| |
817 |
| - | |
| 815 | + | |
818 | 816 |
| |
819 |
| - | |
820 |
| - | |
821 |
| - | |
| 817 | + | |
| 818 | + | |
822 | 819 |
| |
823 | 820 |
| |
824 | 821 |
| |
825 |
| - | |
| 822 | + | |
| 823 | + | |
826 | 824 |
| |
827 | 825 |
| |
828 |
| - | |
829 |
| - | |
830 | 826 |
| |
831 |
| - | |
832 |
| - | |
833 |
| - | |
834 |
| - | |
835 |
| - | |
836 |
| - | |
837 |
| - | |
838 |
| - | |
839 |
| - | |
840 |
| - | |
841 |
| - | |
842 |
| - | |
843 | 827 |
| |
844 | 828 |
| |
845 |
| - | |
846 |
| - | |
| 829 | + | |
847 | 830 |
| |
848 |
| - | |
849 |
| - | |
850 |
| - | |
851 |
| - | |
852 |
| - | |
853 |
| - | |
854 |
| - | |
855 |
| - | |
856 |
| - | |
857 |
| - | |
858 |
| - | |
859 |
| - | |
860 |
| - | |
| 831 | + | |
| 832 | + | |
| 833 | + | |
| 834 | + | |
| 835 | + | |
| 836 | + | |
| 837 | + | |
861 | 838 |
| |
862 | 839 |
| |
863 | 840 |
| |
864 | 841 |
| |
865 | 842 |
| |
866 |
| - | |
867 |
| - | |
| 843 | + | |
868 | 844 |
| |
869 | 845 |
| |
870 | 846 |
| |
| |||
876 | 852 |
| |
877 | 853 |
| |
878 | 854 |
| |
879 |
| - | |
880 | 855 |
| |
881 | 856 |
| |
882 | 857 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
135 | 135 |
| |
136 | 136 |
| |
137 | 137 |
| |
138 |
| - | |
| 138 | + | |
139 | 139 |
| |
140 | 140 |
| |
141 | 141 |
| |
|
0 commit comments
Comments
(0)