| Summary: | parse_message_body inexplicably parses a string it just built. | ||
|---|---|---|---|
| Product: | Ytstenut | Reporter: | Will Thompson <will> |
| Component: | plugins | Assignee: | Jonny Lamb <jonny.lamb> |
| Status: | NEW --- | QA Contact: | |
| Severity: | normal | ||
| Priority: | medium | ||
| Version: | unspecified | ||
| Hardware: | Other | ||
| OS: | All | ||
| Whiteboard: | |||
| i915 platform: | i915 features: | ||
| Attachments: | parse_message_body: clean up empty body special case | ||
Created attachment 51799 [details] [review] parse_message_body: clean up empty body special case I haven't tested my patch, I should admit. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.
static WockyNodeTree * parse_message_body (const gchar *body, GError **error) { // … snip snip … if (body == NULL || *body == '\0') body = "<ytstenut:message xmlns:ytstenut=\"" YTST_MESSAGE_NS "\" />"; and then we go on to parse that message back out. Buh?