Dirk wrote:
Index: ActionHandler.pm
===================================================================
--- ActionHandler.pm    (revision 291)
+++ ActionHandler.pm    (working copy)
@@ -575,7 +575,7 @@
# recover this item within the current parent
     my $parentinfo = $physinfo->{parents}->{$row->{parentphys}};
-    if (undef $parentinfo->{deleted}) {
+    if (!defined $parentinfo->{deleted}) {
         $self->{errmsg} .= "Attempt to recover an active item '$physname':\n"
             . "$self->{physname_seen}\n";

I just noticed this line... regardless of the rest of the patch, you definitely don't want to be doing an undef there; undef will un-define a variable, not test for whether it's defined.

_______________________________________________
vss2svn-users mailing list
Project homepage:
http://www.pumacode.org/projects/vss2svn/
Subscribe/Unsubscribe/Admin:
http://lists.pumacode.org/mailman/listinfo/vss2svn-users-lists.pumacode.org
Mailing list web interface (with searchable archives):
http://dir.gmane.org/gmane.comp.version-control.subversion.vss2svn.user

Reply via email to