[CalendarServer-changes] [285] CalendarServer/trunk/lib-patches/Twisted/twisted.web2.dav.xattrprops. patch

[source_changes at macosforge.org]

Revision: 285
          http://trac.macosforge.org/projects/calendarserver/changeset/285
Author:   cdaboo at apple.com
Date:     2006-10-23 08:12:31 -0700 (Mon, 23 Oct 2006)

Log Message:
-----------
Better performance for encoding property names for xattrs.

Added Paths:
-----------
    CalendarServer/trunk/lib-patches/Twisted/twisted.web2.dav.xattrprops.patch

Added: CalendarServer/trunk/lib-patches/Twisted/twisted.web2.dav.xattrprops.patch
===================================================================
--- CalendarServer/trunk/lib-patches/Twisted/twisted.web2.dav.xattrprops.patch	                        (rev 0)
+++ CalendarServer/trunk/lib-patches/Twisted/twisted.web2.dav.xattrprops.patch	2006-10-23 15:12:31 UTC (rev 285)
@@ -0,0 +1,23 @@
+Index: twisted/web2/dav/xattrprops.py
+===================================================================
+--- twisted/web2/dav/xattrprops.py	(revision 18375)
++++ twisted/web2/dav/xattrprops.py	(working copy)
+@@ -66,16 +66,8 @@
+         deadPropertyXattrPrefix = "user."
+ 
+     def _encode(clazz, name):
+-        #
+-        # FIXME: The xattr API in Mac OS 10.4.2 breaks if you have "/" in an
+-        # attribute name (radar://4202440). We'll quote the strings to get rid
+-        # of "/" characters for now.
+-        #
+-        result = list("{%s}%s" % name)
+-        for i in range(len(result)):
+-            c = result[i]
+-            if c in "%/": result[i] = "%%%02X" % (ord(c),)
+-        r = clazz.deadPropertyXattrPrefix + ''.join(result)
++        result = urllib.quote("{%s}%s" % name, safe='{}:')
++        r = clazz.deadPropertyXattrPrefix + result
+         return r
+ 
+     def _decode(clazz, name):

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.macosforge.org/pipermail/calendarserver-changes/attachments/20061023/89f39404/attachment.html