Two or more consecutive * characters were not handled. A fix was on the web 
site this code comes from.

Before calling wildMatch recursively with "str + 1", the code did not 
check whether *str was zero, in which case the "str + 1" is illegal.

This also changes the names of the function parameters which were not 
consistent in .hpp and .cpp files.

Also: Rename wildMatch to wild_match for consistency with our naming scheme.
You can view, comment on, or merge this pull request online at:

  https://github.com/osm2pgsql-dev/osm2pgsql/pull/2321

-- Commit Summary --

  * Fix two bugs in wildcard match
  * Rename wildMatch() to wild_match()

-- File Changes --

    M src/tagtransform-c.cpp (2)
    M src/wildcmp.cpp (27)
    M src/wildcmp.hpp (2)
    M tests/test-wildcard-match.cpp (49)

-- Patch Links --

https://github.com/osm2pgsql-dev/osm2pgsql/pull/2321.patch
https://github.com/osm2pgsql-dev/osm2pgsql/pull/2321.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/osm2pgsql-dev/osm2pgsql/pull/2321
You are receiving this because you are subscribed to this thread.

Message ID: <osm2pgsql-dev/osm2pgsql/pull/[email protected]>
_______________________________________________
Tile-serving mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/tile-serving

Reply via email to