[Geany-devel] update GNU regex - Re: What's the point of compiling geany without regex support?

[Jiří Techet]

On Fri, Oct 15, 2010 at 14:24, Nick Treleaven
<nick.treleaven at btinternet.com> wrote:
> On Tue, 12 Oct 2010 14:30:00 +0200
> Jiří Techet <techet at gmail.com> wrote:
>
>> >> Attached you'll find a patch that updates the regex library to the
>> >> latest version from glibc. This removes the warnings Lex was
>> >> experiencing on a 64bit machine. I looked at how ctags does it and
>> >> made the patch in a similar way.
>> >
>> > I'm not sure if the glibc version will build on Windows. I don't have a
>> > Windows build environment set up ATM to try it.
>> >
>> > It might be best to use the version from latest CTags. The
>> > gnu_regex/README.txt says:
>> > "Minor changes were made to eliminate compiler errors and warnings."
>> >
>> > Using the same version as CTags could potentially also prevent any
>> > regex-incompatibility bugs with CTags parsers.
>>
>> As far as I remember the CTags version and the glibc version of regex
>> are basically the same.
>
> They're not the same, the diff is significant.

OK, was doing it some time ago so I was probably wrong.

>
> I think we should use the CTags version.
>
>> I also changed some symbol name in the regex
>> sources because of a conflict with some symbol in my system header
>> files.
>
> What's the conflicting symbol?

It was in regex_internal.h - I had to change:

# define __mempcpy mempcpy

to

# define ___mempcpy mempcpy

but I expect CTags fixed this somehow too.

Regards,

Jiri

>
> Regards,
> Nick
> _______________________________________________
> Geany-devel mailing list
> Geany-devel at uvena.de
> http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
>