git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] compat/regex: define out variables only used under RE_ENABLE_I18N

From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date
Aug 19, 2010, 20:31 UTC
Message-ID
<AANLkTikod-o3GHqyFSu7yKJZMc1pgZUQm+pEb-ErAYDB@mail.gmail.com>
In-Reply-To
<7vtymq74mp.fsf@alter.siamese.dyndns.org>
On Thu, Aug 19, 2010 at 20:02, Junio C Hamano <gitster@pobox.com> wrote:
Show 48 quoted lines
> Ævar Arnfjörð Bjarmason  <avarab@gmail.com> writes:
>
>> Wrap variables that were only used RE_ENABLE_I18N in `#ifdef
>> RE_ENABLE_I18N`. This eliminates compiler warnings when compiling with
>> NO_REGEX=YesPlease.
>>
>> Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
>
> Hmm, this unfortunately is not enough for me to compile it locally.
>
> In file included from compat/regex/regex.c:77:
> compat/regex/regcomp.c: In function 're_compile_pattern':
> compat/regex/regcomp.c:237: error: old-style function definition
> compat/regex/regcomp.c: In function 're_set_syntax':
> compat/regex/regcomp.c:276: error: old-style function definition
> compat/regex/regcomp.c: In function 're_compile_fastmap':
> compat/regex/regcomp.c:289: error: old-style function definition
> compat/regex/regcomp.c: In function 'regcomp':
> compat/regex/regcomp.c:491: error: old-style function definition
> compat/regex/regcomp.c: In function 'regfree':
> compat/regex/regcomp.c:655: error: old-style function definition
> In file included from compat/regex/regex.c:78:
> compat/regex/regexec.c: In function 'regexec':
> compat/regex/regexec.c:227: error: old-style function definition
> compat/regex/regexec.c: In function 're_match':
> compat/regex/regexec.c:311: error: old-style function definition
> compat/regex/regexec.c: In function 're_search':
> compat/regex/regexec.c:324: error: old-style function definition
> compat/regex/regexec.c: In function 're_match_2':
> compat/regex/regexec.c:337: error: old-style function definition
> compat/regex/regexec.c: In function 're_search_2':
> compat/regex/regexec.c:351: error: old-style function definition
> compat/regex/regexec.c: In function 're_search_2_stub':
> compat/regex/regexec.c:366: error: old-style function definition
> compat/regex/regexec.c: In function 're_search_stub':
> compat/regex/regexec.c:410: error: old-style function definition
> compat/regex/regexec.c: In function 're_copy_regs':
> compat/regex/regexec.c:497: error: old-style function definition
> compat/regex/regexec.c: In function 're_set_registers':
> compat/regex/regexec.c:578: error: old-style function definition
> compat/regex/regexec.c: In function 're_search_internal':
> compat/regex/regexec.c:631: error: old-style function definition
> compat/regex/regexec.c: In function 'prune_impossible_nodes':
> compat/regex/regexec.c:952: error: old-style function definition
> make: *** [compat/regex/regex.o] Error 1
>
> The fix should be straightforward, and I think it should be squashed with
> the header ANSIfication you sent earlier.

That's odd. The old regex library only used k&r style function declerations *except* for regerror, and only then because a variable name conflicted with a Windows typedef.

Could you compile the old compat/regex library on the same system? What compiler/version/flags are these?

Previous: Junio C HamanoNext: Junio C Hamano
Message 12 of 23 in “What's cooking in git.git (Aug 2010, #04; Wed, 18)”
  1. Junio C HamanoAug 18, 2010
  2. Jonathan NiederAug 19, 2010
  3. Johannes SixtAug 19, 2010
  4. Ævar Arnfjörð BjarmasonAug 19, 2010
  5. Ævar Arnfjörð BjarmasonAug 19, 2010
  6. jn/update-contrib-example-merge in puJonathan Nieder, Aug 19, 2010
  7. Ævar Arnfjörð BjarmasonAug 19, 2010
  8. Ævar Arnfjörð BjarmasonAug 19, 2010
  9. Aharon RobbinsAug 19, 2010
  10. compat/regex: define out variables only used under RE_ENABLE_I18NÆvar Arnfjörð Bjarmason, Aug 19, 2010
  11. Junio C HamanoAug 19, 2010
  12. Ævar Arnfjörð BjarmasonAug 19, 2010
  13. Junio C HamanoAug 19, 2010
  14. Ævar Arnfjörð BjarmasonAug 19, 2010
  15. compat/regex: get rid of old-style definitionJunio C Hamano, Aug 19, 2010
  16. Ævar Arnfjörð BjarmasonAug 20, 2010
  17. Karsten BleesAug 25, 2010
  18. Fix compat/regex ANSIfication on MinGWJohannes Sixt, Aug 26, 2010
  19. Ævar Arnfjörð BjarmasonAug 26, 2010
  20. karsten.blees@dcon.deSep 3, 2010
  21. Ævar Arnfjörð BjarmasonSep 3, 2010
  22. Junio C HamanoSep 4, 2010
  23. Johannes SixtSep 7, 2010

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.