Closed mapserver-bot closed 8 years ago
Author: dmorissette Date: 2004/10/12 - 20:10
My 0.02$:
I think only #1 and #3 would be viable options for the future, with #3 preferred
of course if we can find a thread-safe parser. About #2, I would be worried that
if we do that we could forget to put locking around new code or MapScript
methods that may not be calling msEvalExpression() today but that may end up
calling it later down the road.
Author: sgillies@frii.com Date: 2004/10/13 - 05:46
Agreed, #2 is no good. Many potential problems there.
I'm actually quite happy with our present position. I try to avoid heavy
use of mapserver logical expressions anyway. A second lexer or rewrite
of the expression parser is low priority for me.
Author: sdlime Date: 2004/10/13 - 06:34
You'll need a parser for logical expressions regardless. If we go to XML
mapfiles with a different parser on that end then only bison/flex will be needed
for expressions and this will be an easier change. Flex can generate a
re-entrant lexer and bison can create a threadsafe parser, but that sort of a
switch is 5.x and is affected by other things. Mapfile.c contains the mapfile
parser, it's the lexer that's doubled up. Could have two of those buried in a
mapObj I suppose.
I agree with Dan that there are bigger fish to fry at the moment.
Steve
Author: dmorissette Date: 2004/10/13 - 16:04
It seems that we all agree that this won't be addressed in 4.4.
I have created a new "FUTURE" target milestone to mark bugs that should be
revisited and possibly scheduled in a future release and I have set target
milestone FUTURE for this bug.
Author: sgillies@frii.com Date: 2006/06/02 - 04:25
reassigning.
Author: unicoletti Date: 2007/08/09 - 09:56 '''Bookmark''' Notes on how to enable the reentrant feature (thread safe) of flex:
This issue has been closed due to lack of activity. This doesn't mean the issue is invalid, it simply got no attention within the last year. Please reopen with missing/relevant information if still valid.
Typically, issues fall in this state for one of the following reasons:
Reporter: dmorissette Date: 2004/10/12 - 20:09 Trac URL: http://trac.osgeo.org/mapserver/ticket/955