Commit 2e4a72ed authored by Simon McVittie's avatar Simon McVittie

desktop-file: Justify implementation of is_valid_section_name()

Signed-off-by: Simon McVittie's avatarSimon McVittie <smcv@collabora.com>
(cherry picked from commit 99580298)
parent 2b099429
......@@ -378,7 +378,12 @@ parse_comment_or_blank (BusDesktopFileParser *parser)
static dbus_bool_t
is_valid_section_name (const char *name)
{
/* 5. Group names may contain all ASCII characters except for control characters and '[' and ']'. */
/* 5. Group names may contain all ASCII characters except for control characters and '[' and ']'.
*
* We don't use isprint() here because it's locale-dependent. ASCII
* characters <= 0x1f and 0x7f are control characters, and bytes with
* values >= 0x80 aren't ASCII. 0x20 is a space, which we must allow,
* not least because DBUS_SERVICE_SECTION contains one. */
while (*name)
{
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment