Commit f77a3951 authored by Lars Ingebrigtsen's avatar Lars Ingebrigtsen
Browse files

Clarify Freplace_match logic

* src/search.c (Freplace_match): Add some doc strings to clarify
the logic and do a minor clean up (bug#19208).
parent 6c7ab768
......@@ -2397,11 +2397,15 @@ since only regular expressions have distinguished subexpressions. */)
else
{
CHECK_FIXNUM (subexp);
if (! (0 <= XFIXNUM (subexp) && XFIXNUM (subexp) < search_regs.num_regs))
args_out_of_range (subexp, make_fixnum (search_regs.num_regs));
sub = XFIXNUM (subexp);
/* Sanity check to see whether the subexp is larger than the
allowed number of sub-regexps. */
if (sub >= 0 && sub > search_regs.num_regs)
args_out_of_range (subexp, make_fixnum (search_regs.num_regs));
}
/* Sanity check to see whether the text to replace is present in the
buffer/string. */
if (NILP (string))
{
if (search_regs.start[sub] < BEGV
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment