style: Put closing bracket and else in one line please (always `} else {`)

If we are on a tag that can not be translated, we really shouldn't set `tag_id` to the localized identifier, but emit a warning and do `tag_id = g_strdup (node_name);`, so: ```c if (translatable) { tag_id = g_strdup_printf ("%s (lang=%s)", node_name, lang); } else { tag_id = g_strdup (node_name); as_validator_add_issue (validator, node, "unsupported-translation", node_name); } ``` Otherwise we'll not emit some important warnings at first go in case a file has multiple errors.

style: This *does* need braces now, as the following `else` branch also has them

nit: extra space not needed

style: Single-statement if clause shouldn't have extra braces.

style: Please use snake_case => `allow_translations` - but for conciseness, I'd probably name this `translatable` or `allow_l10n` (your choice though) :)

typo: better: `This tag is not translatable` (everything is per the specification, so we usually omit overly verbose wording)

naming: better: `translation-unsupported` or `tag-not-translatable`