PCRE dependency

Forum for general chit-chat or off-topic discussion.

Moderators: CEGUI MVP, CEGUI Team

Pompei2
Home away from home
Home away from home
Posts: 489
Joined: Tue May 23, 2006 16:31

PCRE dependency

Postby Pompei2 » Sat Nov 01, 2008 18:50

Hello,

As far as I know, you use PCRE only for the input mask of the editboxes, or am I missing something (grep'ed the sources) ?

Is there any easy way of getting rid of that dependency, as I don't need it ??

I may modify the sources to no more use it when a define is set, or even to use only expressions with "*" and "?" as metacharacters. Would somebody insert it into trunk ? I am not able to edit the configure script, neither the VisualC++ project files as I don't know enough about them.

Or has anybody else done this before (So I do not need to reinvent the wheel) ?

User avatar
CrazyEddie
CEGUI Project Lead
Posts: 6760
Joined: Wed Jan 12, 2005 12:06
Location: England
Contact:

Postby CrazyEddie » Tue Nov 04, 2008 10:07

Hi,

As far as I know nobody has done this before. You are correct that PCRE is only used for string validation / input masking on editboxes and spinner widgets.

We intend to introduce a more granular compile with more compile options to discard unwanted parts and also dependencies (including PCRE). This decision was in response to another users request / suggestion.

There will be a ticket added for this (and some other requested options) once I get around to adding them on Mantis. In the mean time, if you want to make your mods and submit them as a patch (on Mantis, I'll tie it all up once I add the feature request tickets), we'll likely use some or all of it when implementing the changes mentioned above.

CE.

Pompei2
Home away from home
Home away from home
Posts: 489
Joined: Tue May 23, 2006 16:31

Postby Pompei2 » Sat Nov 15, 2008 22:35

CrazyEddie wrote:.. and spinner widgets.


Hmm, I took a quick look at that and must admit that it doesn't seem as easy as I tought before, I forgot the spinner :)

I'll look more into detail at this in some time. I will report my success/failure here for information.

User avatar
CrazyEddie
CEGUI Project Lead
Posts: 6760
Joined: Wed Jan 12, 2005 12:06
Location: England
Contact:

Postby CrazyEddie » Sun Nov 16, 2008 12:44

A thought I had earlier this morning is that it might be useful to abstract our use of PCRE, thus enabling the use of other RegEx libs and user supplied 'lite' or do-nothing versions where appropriate (which might be a better option than excluding widgets entirely due to their reliance on RegEx).

CE

Pompei2
Home away from home
Home away from home
Posts: 489
Joined: Tue May 23, 2006 16:31

Postby Pompei2 » Sun Nov 16, 2008 15:28

CrazyEddie wrote:A thought I had earlier this morning is that it might be useful to abstract our use of PCRE, thus enabling the use of other RegEx libs and user supplied 'lite' or do-nothing versions where appropriate


Yeah, that is a very good idea that I will have in mind when i will work on that thing. Thanks for telling me.

Sairon
Not too shy to talk
Not too shy to talk
Posts: 22
Joined: Sun Dec 07, 2008 01:11

Postby Sairon » Mon Dec 08, 2008 22:10

I think it would be nice with an option to use boost regex lib, since a lot of people most likely already will have a dependency on boost.

User avatar
CrazyEddie
CEGUI Project Lead
Posts: 6760
Joined: Wed Jan 12, 2005 12:06
Location: England
Contact:

Postby CrazyEddie » Tue Dec 09, 2008 09:18

Sairon wrote:I think it would be nice with an option to use boost regex lib, since a lot of people most likely already will have a dependency on boost.

I agree that once abstracted, the CEGUI should definitely support a boost regex option. This is actually highly amusing to me, because originally we were using boost regex and removed it in favour of PCRE (mainly due to anti-boost user requests) :)

Sairon
Not too shy to talk
Not too shy to talk
Posts: 22
Joined: Sun Dec 07, 2008 01:11

Postby Sairon » Tue Dec 09, 2008 23:24

CrazyEddie wrote:
Sairon wrote:I think it would be nice with an option to use boost regex lib, since a lot of people most likely already will have a dependency on boost.

I agree that once abstracted, the CEGUI should definitely support a boost regex option. This is actually highly amusing to me, because originally we were using boost regex and removed it in favour of PCRE (mainly due to anti-boost user requests) :)


Hehe, that's amusing. I find boost to be a lot easier to setup and maintain than PCRE, at least on windows hehe

Pompei2
Home away from home
Home away from home
Posts: 489
Joined: Tue May 23, 2006 16:31

Postby Pompei2 » Thu Dec 11, 2008 18:01

But boost is a quite big dependency ...


Return to “Offtopic Discussion”

Who is online

Users browsing this forum: No registered users and 6 guests