Skip to content

Commit

Permalink
compat/regex: get rid of old-style definition
Browse files Browse the repository at this point in the history
These files mostly used ANSI style function definitions, but with small
number of old-style ones.  Convert them to consistently use ANSI style.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Junio C Hamano committed Aug 19, 2010
1 parent b50f370 commit 178b331
Show file tree
Hide file tree
Showing 2 changed files with 55 additions and 69 deletions.
23 changes: 9 additions & 14 deletions compat/regex/regcomp.c
Original file line number Diff line number Diff line change
Expand Up @@ -230,10 +230,9 @@ btowc (int c)
are set in BUFP on entry. */

const char *
re_compile_pattern (pattern, length, bufp)
const char *pattern;
size_t length;
struct re_pattern_buffer *bufp;
re_compile_pattern (const char *pattern,
size_t length,
struct re_pattern_buffer *bufp)
{
reg_errcode_t ret;

Expand Down Expand Up @@ -271,8 +270,7 @@ reg_syntax_t re_syntax_options;
defined in regex.h. We return the old syntax. */

reg_syntax_t
re_set_syntax (syntax)
reg_syntax_t syntax;
re_set_syntax (reg_syntax_t syntax)
{
reg_syntax_t ret = re_syntax_options;

Expand All @@ -284,8 +282,7 @@ weak_alias (__re_set_syntax, re_set_syntax)
#endif

int
re_compile_fastmap (bufp)
struct re_pattern_buffer *bufp;
re_compile_fastmap (struct re_pattern_buffer *bufp)
{
re_dfa_t *dfa = (re_dfa_t *) bufp->buffer;
char *fastmap = bufp->fastmap;
Expand Down Expand Up @@ -484,10 +481,9 @@ re_compile_fastmap_iter (regex_t *bufp, const re_dfastate_t *init_state,
the return codes and their meanings.) */

int
regcomp (preg, pattern, cflags)
regex_t *__restrict preg;
const char *__restrict pattern;
int cflags;
regcomp (regex_t *__restrict preg,
const char *__restrict pattern,
int cflags)
{
reg_errcode_t ret;
reg_syntax_t syntax = ((cflags & REG_EXTENDED) ? RE_SYNTAX_POSIX_EXTENDED
Expand Down Expand Up @@ -650,8 +646,7 @@ free_dfa_content (re_dfa_t *dfa)
/* Free dynamically allocated space used by PREG. */

void
regfree (preg)
regex_t *preg;
regfree (regex_t *preg)
{
re_dfa_t *dfa = (re_dfa_t *) preg->buffer;
if (BE (dfa != NULL, 1))
Expand Down
101 changes: 46 additions & 55 deletions compat/regex/regexec.c
Original file line number Diff line number Diff line change
Expand Up @@ -218,12 +218,12 @@ static reg_errcode_t extend_buffers (re_match_context_t *mctx)
We return 0 if we find a match and REG_NOMATCH if not. */

int
regexec (preg, string, nmatch, pmatch, eflags)
const regex_t *__restrict preg;
const char *__restrict string;
size_t nmatch;
regmatch_t pmatch[];
int eflags;
regexec (
const regex_t *__restrict preg,
const char *__restrict string,
size_t nmatch,
regmatch_t pmatch[],
int eflags)
{
reg_errcode_t err;
int start, length;
Expand Down Expand Up @@ -303,11 +303,11 @@ compat_symbol (libc, __compat_regexec, regexec, GLIBC_2_0);
match was found and -2 indicates an internal error. */

int
re_match (bufp, string, length, start, regs)
struct re_pattern_buffer *bufp;
const char *string;
int length, start;
struct re_registers *regs;
re_match (struct re_pattern_buffer *bufp,
const char *string,
int length,
int start,
struct re_registers *regs)
{
return re_search_stub (bufp, string, length, start, 0, length, regs, 1);
}
Expand All @@ -316,11 +316,10 @@ weak_alias (__re_match, re_match)
#endif

int
re_search (bufp, string, length, start, range, regs)
struct re_pattern_buffer *bufp;
const char *string;
int length, start, range;
struct re_registers *regs;
re_search (struct re_pattern_buffer *bufp,
const char *string,
int length, int start, int range,
struct re_registers *regs)
{
return re_search_stub (bufp, string, length, start, range, length, regs, 0);
}
Expand All @@ -329,11 +328,10 @@ weak_alias (__re_search, re_search)
#endif

int
re_match_2 (bufp, string1, length1, string2, length2, start, regs, stop)
struct re_pattern_buffer *bufp;
const char *string1, *string2;
int length1, length2, start, stop;
struct re_registers *regs;
re_match_2 (struct re_pattern_buffer *bufp,
const char *string1, int length1,
const char *string2, int length2, int start,
struct re_registers *regs, int stop)
{
return re_search_2_stub (bufp, string1, length1, string2, length2,
start, 0, regs, stop, 1);
Expand All @@ -343,11 +341,10 @@ weak_alias (__re_match_2, re_match_2)
#endif

int
re_search_2 (bufp, string1, length1, string2, length2, start, range, regs, stop)
struct re_pattern_buffer *bufp;
const char *string1, *string2;
int length1, length2, start, range, stop;
struct re_registers *regs;
re_search_2 (struct re_pattern_buffer *bufp,
const char *string1, int length1,
const char *string2, int length2, int start,
int range, struct re_registers *regs, int stop)
{
return re_search_2_stub (bufp, string1, length1, string2, length2,
start, range, regs, stop, 0);
Expand All @@ -357,12 +354,11 @@ weak_alias (__re_search_2, re_search_2)
#endif

static int
re_search_2_stub (bufp, string1, length1, string2, length2, start, range, regs,
stop, ret_len)
struct re_pattern_buffer *bufp;
const char *string1, *string2;
int length1, length2, start, range, stop, ret_len;
struct re_registers *regs;
re_search_2_stub (struct re_pattern_buffer *bufp,
const char *string1, int length1,
const char *string2, int length2, int start,
int range, struct re_registers *regs,
int stop, int ret_len)
{
const char *str;
int rval;
Expand Down Expand Up @@ -402,11 +398,10 @@ re_search_2_stub (bufp, string1, length1, string2, length2, start, range, regs,
otherwise the position of the match is returned. */

static int
re_search_stub (bufp, string, length, start, range, stop, regs, ret_len)
struct re_pattern_buffer *bufp;
const char *string;
int length, start, range, stop, ret_len;
struct re_registers *regs;
re_search_stub (struct re_pattern_buffer *bufp,
const char *string, int length, int start,
int range, int stop,
struct re_registers *regs, int ret_len)
{
reg_errcode_t result;
regmatch_t *pmatch;
Expand Down Expand Up @@ -490,10 +485,9 @@ re_search_stub (bufp, string, length, start, range, stop, regs, ret_len)
}

static unsigned
re_copy_regs (regs, pmatch, nregs, regs_allocated)
struct re_registers *regs;
regmatch_t *pmatch;
int nregs, regs_allocated;
re_copy_regs (struct re_registers *regs,
regmatch_t *pmatch,
int nregs, int regs_allocated)
{
int rval = REGS_REALLOCATE;
int i;
Expand Down Expand Up @@ -570,11 +564,11 @@ re_copy_regs (regs, pmatch, nregs, regs_allocated)
freeing the old data. */

void
re_set_registers (bufp, regs, num_regs, starts, ends)
struct re_pattern_buffer *bufp;
struct re_registers *regs;
unsigned num_regs;
regoff_t *starts, *ends;
re_set_registers (struct re_pattern_buffer *bufp,
struct re_registers *regs,
unsigned num_regs,
regoff_t *starts,
regoff_t *ends)
{
if (num_regs)
{
Expand Down Expand Up @@ -621,13 +615,11 @@ re_exec (s)
(START + RANGE >= 0 && START + RANGE <= LENGTH) */

static reg_errcode_t
re_search_internal (preg, string, length, start, range, stop, nmatch, pmatch,
eflags)
const regex_t *preg;
const char *string;
int length, start, range, stop, eflags;
size_t nmatch;
regmatch_t pmatch[];
re_search_internal (const regex_t *preg,
const char *string,
int length, int start, int range, int stop,
size_t nmatch, regmatch_t pmatch[],
int eflags)
{
reg_errcode_t err;
const re_dfa_t *dfa = (const re_dfa_t *) preg->buffer;
Expand Down Expand Up @@ -947,8 +939,7 @@ re_search_internal (preg, string, length, start, range, stop, nmatch, pmatch,
}

static reg_errcode_t
prune_impossible_nodes (mctx)
re_match_context_t *mctx;
prune_impossible_nodes (re_match_context_t *mctx)
{
const re_dfa_t *const dfa = mctx->dfa;
int halt_node, match_last;
Expand Down

0 comments on commit 178b331

Please sign in to comment.