Hi Carlos,

I think one test is enough, and your change doesn't seem to catch "fix_me".
Try the attached version of the patch.
The regex '(?i)fix[ _]?+me'
uses a so called "Possessive quantifier" :
X?+     means X, once or not at all

Gerd


________________________________________
Von: [email protected] 
<[email protected]> im Auftrag von Carlos Dávila 
<[email protected]>
Gesendet: Sonntag, 10. Januar 2016 15:05
An: Development list for mkgmap
Betreff: [mkgmap-dev] Catch more fixme variants to delete

Default style contains some rules in inc/name to catch fixme's in name,
ref, brand and operator tags and delete them, but the number of variants
of fixme's is quite wide and some are not catched, so I have replaced
current rules with a regular expression, to try to catch them all.
Attached is a patch that implement this.

Attachment: delete_fixme_v2.patch
Description: delete_fixme_v2.patch

_______________________________________________
mkgmap-dev mailing list
[email protected]
http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev

Reply via email to