[MapServer C Library] Include parsing not closing Symbols (?)

[jmckenna@…]

Assefa discovered an interesting one (bare with me as I try to explain it, as I
bet at first you'll think it is a mapscript issue I'm sure):

- in PHP/Mapscript, a loop that simply creates a map object, with a mapfile that
has an include, AND has a symbol file, fails
- the odd thing is that if you add an extra END in the symbol file it does not
fail anymore (?)

Here is a tiny test case (http://www2.dmsolutions.ca/netbin/include-bug.zip) 1KB
  - run test.php

(i'm not sure if this is related to bug#2013 - font handling with includes)

PHP Script
==========

<?php
if (!extension_loaded("MapScript")) 
     dl("php_mapscript.dll");

for ($i=0; $i<4; $i++)
{
  $oSourceMap = ms_newmapobj('./test.map');
  if ($oSourceMap)
    echo "success <br>\n";
  else
    echo "failure <br>\n";
}
?>

Mapfile
=======

NAME include-test
STATUS ON
SIZE 400 300
SYMBOLSET ./symbols.sym
EXTENT -180 -90 180 90
UNITS DD
IMAGECOLOR 255 255 255

DEBUG on

WEB
  IMAGEPATH "/ms4w/tmp/ms_tmp/" 
  IMAGEURL "/ms_tmp/"
END

INCLUDE "./include.lay"

END # Map File

Include File
============

LAYER
  NAME test
  TYPE POINT
  STATUS ON
  FEATURE
    POINTS
       69.43 35.15
    END
  END
  CLASS
    STYLE
      COLOR 0 0 250
      SYMBOL 'circle'
      SIZE 100
    END
  END
END # layer

Symbol File
===========

Symbol
  Name 'circle'
  Type ELLIPSE
  Filled TRUE
  Points
    1 1
  END
END

Testing Environment
===================

MapServer 4.10.1
windows

[jmckenna@…]

no response.  Did I not explain this clear enough?

[sdlime]

This is the first I've heard of the bug, original message never made it so I 
haven't had a chance to look yet.

Steve

[dmorissette]

Jeff, Assefa, would you be able to test this again with the latest beta? Some fixes have been done in that area in 5.0 and we'd need to know whether this is still an issue.

[sdlime]

I'll wait until there is confirmation...

Steve

[assefa]

It seemed to be fixed for me using MapServer svn using the test data.

Jeff please confirm and close.

[sdlime]

Moving to 5.0.1 without test confirmation...

Steve

[jmckenna]

Tested successfully (no problems) with mapserver-5.0-rc2.