Merge lp:~ted/libdbusmenu/bigger-max-xml into lp:libdbusmenu/0.5

Proposed by Ted Gould
Status: Merged
Merged at revision: 151
Proposed branch: lp:~ted/libdbusmenu/bigger-max-xml
Merge into: lp:libdbusmenu/0.5
Diff against target: 12 lines (+2/-1)
1 file modified
libdbusmenu-glib/client.c (+2/-1)
To merge this branch: bzr merge lp:~ted/libdbusmenu/bigger-max-xml
Reviewer Review Type Date Requested Status
Neil J. Patel (community) Approve
Review via email: mp+33219@code.launchpad.net

Description of the change

Increase the max number of characters the XML string can be. GIMP is crazy.

To post a comment you must log in.
Revision history for this message
Neil J. Patel (njpatel) wrote :

Wheee!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'libdbusmenu-glib/client.c'
2--- libdbusmenu-glib/client.c 2010-08-16 15:59:53 +0000
3+++ libdbusmenu-glib/client.c 2010-08-20 14:27:43 +0000
4@@ -1225,7 +1225,8 @@
5
6 xmlDocPtr xmldoc;
7
8- xmldoc = xmlReadMemory(layout, g_utf8_strlen(layout, 16*1024), "dbusmenu.xml", NULL, 0);
9+ /* No one should need more characters than this! */
10+ xmldoc = xmlReadMemory(layout, g_utf8_strlen(layout, 1024*1024), "dbusmenu.xml", NULL, 0);
11
12 xmlNodePtr root = xmlDocGetRootElement(xmldoc);
13

Subscribers

People subscribed via source and target branches