Hi all,

attached is 2nd version of the patch. Steves test showed an error in my 1st patch.
I've also added tests for the part function.

I noticed that it is currently not possible to use ":" as a separator in the part style function.
Seems not to be needed?

Gerd

Date: Tue, 16 Dec 2014 09:22:05 +0000
From: steve@parabola.me.uk
To: mkgmap-dev@lists.mkgmap.org.uk
Subject: Re: [mkgmap-dev] [Patch v1] check length in substring style function

Hi Gerd

> I forgot to answer this post from gbbickerton:
> "May i suggest that a test is made to check that the string is long
> enough before applying this filter otherwise an exception occurs .It
> would be unsafe to use this filter in any style without such a test. "

Well remembered, thanks.

It could also do with some tests, I've attached some.

I would like to use SyntaxException instead of ExitException for
all cases where an error can be detected at compile time
in a style file.

I realise that ExitException was already used in that class.
PartFilter is the only other example I can see where it does
not use SyntaxException.

..Steve


_______________________________________________ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev