GNOME Bugzilla – Bug 730181
failed to parse '%' character in gst_structure_from_string
Last modified: 2018-11-03 12:20:35 UTC
gst_structure_from_string seems not to accept '%' character due to missing define in ASCII_IS_STRING in gst_private.h Is that omitted on purpose? 154 /* used in gstvalue.c and gststructure.c */ 155 #define GST_ASCII_IS_STRING(c) (g_ascii_isalnum((c)) || ((c) == '_') || \ 156 ((c) == '-') || ((c) == '+') || ((c) == '/') || ((c) == ':') || \ 157 ((c) == '.'))
Created attachment 276601 [details] [review] Add '%' character in GST_ASCII_IS_STRING macro I add '%' character in GST_ASCII_IS_STRING for the escaped character and it works well.
FWIW, it works if you escape the % with a backslash \ It's probably ommitted on purpose, but I'm not aware of any special meaning of the % character. So it can probably be changed safely if there's a good reason to do so. What are you using the % for though?
> What are you using the % for though? I just found this error when I used html_escaped characters for value, and no more. e.g. a blank character (space) is converted into %20. I think it would be better to support '%' if it has no special purpose :)
I think this is still valid for the urlencoded strings use-case.
-- GitLab Migration Automatic Message -- This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity. You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gstreamer/issues/57.