Index: /branches/release-41/lib/MT/Upgrade.pm
===================================================================
--- /branches/release-41/lib/MT/Upgrade.pm (revision 2637)
+++ /branches/release-41/lib/MT/Upgrade.pm (revision 2660)
@@ -1011,4 +1011,5 @@
         next TYPE if !$class->has_meta();  # nothing to upgrade
 
+        # TODO: what is this supposed to mean?
         my $t = $type;
         my $class_type = $class->properties->{class_type};
@@ -1021,4 +1022,5 @@
             # If there's no appropriate superclass, go to update with the class
             # we have, not the class we want.
+            # TODO: if we're checking against the registry type without modifying it, why are we looking for a better object class first?
             next TYPE if $added_step{$type};  # already got that one
         }
@@ -1027,5 +1029,7 @@
         }
 
+        # TODO: why is this no longer the representative type for a table? that makes us upgrade tables multiple times
         my %step_param = ( type => $t );
+        # TODO: can't we just make this a different upgrade step?
         $step_param{plugindata} = 1
             if ( $class eq 'MT::Category' ) || ( $class eq 'MT::Folder' );
@@ -1125,6 +1129,4 @@
     my $stmt = $dbd->sql_class->new;
 
-    # assumes 'meta' is the meta column name; should be for all core types
-    # we are processing
     my $meta_col = $param{meta_column} || 'meta';