Commit f38a029
committed
ParseXS: refactor: fetch_para(): simplify logic
Simplify the logic of the code which handles C-preprocessor directives
where there's not the (usually required) blank line directly before and
after an XSUB; e.g.
#if X
int foo(...)
...
#endif
This commit splits the logic into two halves: the first is responsible
for handling conditionals directly *before* the XSUB, and the other half
for ones during and directly after the XSUB. Then the second half is
further simplified.
I think this makes the code easier to understand.
I *think* the new code is logically equivalent.1 parent 0028ecc commit f38a029
1 file changed
+14
-20
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
735 | 735 | | |
736 | 736 | | |
737 | 737 | | |
738 | | - | |
739 | | - | |
740 | | - | |
741 | | - | |
742 | | - | |
743 | | - | |
744 | | - | |
745 | | - | |
746 | | - | |
747 | | - | |
748 | | - | |
749 | | - | |
750 | | - | |
751 | | - | |
752 | | - | |
753 | | - | |
| 738 | + | |
| 739 | + | |
| 740 | + | |
| 741 | + | |
754 | 742 | | |
755 | 743 | | |
756 | | - | |
757 | | - | |
758 | | - | |
759 | | - | |
| 744 | + | |
| 745 | + | |
| 746 | + | |
| 747 | + | |
| 748 | + | |
| 749 | + | |
| 750 | + | |
| 751 | + | |
| 752 | + | |
| 753 | + | |
760 | 754 | | |
761 | 755 | | |
762 | 756 | | |
| |||
0 commit comments