[Github-comments] [geany/geany] TM cleanups (#957)

Colomban Wendling notifications at xxxxx
Wed May 4 17:33:47 UTC 2016


> +{
> +	TMParserMap *map = &parser_map[lang];
> +	guint i;
> +
> +	for (i = 0; i < map->size; i++)
> +	{
> +		TMParserMapEntry *entry = &map->entries[i];
> +
> +		if (entry->type == type)
> +			return entry->kind;
> +	}
> +	return '-';
> +}
> +
> +
> +void tm_parser_verify_type_mappings(void)

I was thinking about a separate test executable linked to *libtagmanager.la* and doing just that: https://github.com/b4n/geany/commit/2453bc853bf4716ba41601fe037337ef9cc582f8
If this doesn't lead to any building issues (I need to check if linking a `.la` to both an executable and a library doesn't require building each file both as relocatable and not on some archs, which would blow compile times off the roof, like in early libtool switch), it's fairly trivial and self-contained I think.

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/957/files/aea7d7845917843dd6b02e5a8dc9031cabc87a86..5d9376ea801a6e6c3ec22748741362a3a2eff7ac#r62080852
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160504/46399818/attachment.html>


More information about the Github-comments mailing list