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

[Colomban Wendling]

> +{
> +	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)

> And one more thing - I was thinking about moving the TM part into Geany directory […]. Then TM would depend on Geany and it wouldn't be possible to create a Geany-independent binary.

Should still be possible to have a separate binary for checking, so long as the symbol is public.  We could have either that function public, or a `geany_test_main()` that calls a bunch of tests and can be run without a GUI.  Or something like @codebrainz's idea of something like a self-test option, `--test` that would perform tests and exit (with out without GUI, one could probably be emulated somehow with some weird X-faking thingie).

Anyway, can be done later, but really having tests run with the test suite would be great.

---
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#r66290649
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160608/6eb3b269/attachment.html>