Remove pre-1.0 "svn:this_dir" support?

While working through the various mods I have accumulated over the
last few months I found this one:

Index: subversion/libsvn_wc/entries.c
===================================================================
--- subversion/libsvn_wc/entries.c (revision 21531)
+++ subversion/libsvn_wc/entries.c (working copy)
@@ -469,12 +469,6 @@
 
   /* Find the name and set up the entry under that name. */
   name = apr_hash_get(atts, SVN_WC__ENTRY_ATTR_NAME, APR_HASH_KEY_STRING);
- /* XXX Replace the obsolete "svn:this_dir".
- XXX This code should go away by 1.0 */
- {
- if (name && strcmp(name, "svn:this_dir") == 0)
- name = SVN_WC_ENTRY_THIS_DIR;
- }
   entry->name = name ? apr_pstrdup(pool, name) : SVN_WC_ENTRY_THIS_DIR;
 
   /* Attempt to set revision (resolve_to_defaults may do it later, too) */

This removes support for working copies created by 0.32 clients.
Either the code should go, as per the comment, or the comment should
get modified. Which do people prefer?

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org