diff options
author | David Mitchell <davem@iabyn.com> | 2017-07-24 14:19:29 +0100 |
---|---|---|
committer | David Mitchell <davem@iabyn.com> | 2017-07-27 11:30:21 +0100 |
commit | 55922dbec2982825f2821bdc2dae4ec2b4cb5cd2 (patch) | |
tree | af1bc319a5b24c2f3ec7ab6879b6cc9d733c1739 /regcomp.h | |
parent | df6b4bd56551f2d39f7c0019c23f27181d8c39c4 (diff) | |
download | perl-55922dbec2982825f2821bdc2dae4ec2b4cb5cd2.tar.gz |
RX_FOO(prog) becomes RX_FOO(rx_sv)
There are a family of RX_() macros which take an SV of type REGEXP as an
argument. For historical reasons (regexeps didn't use to be SVs), the name
of the parameter is 'prog', which is also by convention used to name the
actual regexp struct in some places. So e.g. at the top of
re_intuit_start(), you see
struct regexp *const prog = ReANY(rx);
This is confusing. So for this class of macro, rename the parameter from
'prog' to 'rx_sv'. This makes it clearer that the arg should be a REGEXP*
rather than an regexp*.
Note that there are some RXp_() macros which do take a regexp* as an arg;
I've left their parameter name as 'prog'.
Diffstat (limited to 'regcomp.h')
0 files changed, 0 insertions, 0 deletions