The new pack function should get intensively tested. It should be the default one use

all the time. There is a performance problem (it's a lot slower than the optimized versions)
but otherwise it will never get tested intensively.

This commit was SVN r6467.

ompi/datatype/dt_pack.c

@@ -771,7 +771,6 @@ ompi_convertor_prepare_for_send( ompi_convertor_t* convertor,
     convertor->fAdvance = ompi_convertor_pack_homogeneous_with_memcpy;
     convertor->fAdvance = ompi_convertor_pack_no_conversion;
     
-    convertor->fAdvance = ompi_convertor_generic_simple_pack;
     if( datatype->flags & DT_FLAG_CONTIGUOUS ) {
         convertor->flags |= DT_FLAG_CONTIGUOUS;
         if( ((datatype->ub - datatype->lb) == (long)datatype->size)
@@ -780,7 +779,8 @@ ompi_convertor_prepare_for_send( ompi_convertor_t* convertor,
         else
             convertor->fAdvance = ompi_convertor_pack_no_conv_contig_with_gaps;
     }
-    
+    convertor->fAdvance = ompi_convertor_generic_simple_pack;
+
     return OMPI_SUCCESS;
 }