Resharper recomienda cambiar CompareTo a CompareOrdinal

Question

Tengo un clasificador que realiza varias comparaciones. Resharper dice que debería cambiar de String.CompareTo a String.CompareOrdinal. ¿Esto realmente proporciona muchos beneficios o es algo para lo que debo ajustar las reglas de inspección?

CompareTo:

config.Sort(delegate(configData data1, configData data2) 
{ 
    if (data1.SettingName.Equals(data2.SettingName)) 
    { 
     string m1 = data1.SettingMachine; 
     string m2 = data2.SettingMachine; 
     if (m1 == null) 
      m1 = string.Empty; 
     if (m2 == null) 
      m2 = string.Empty; 
     return m1.CompareTo(m2); 
    } 

    return data1.SettingName.CompareTo(data2.SettingName); 
}); 

CompareOrdinal:

config.Sort(delegate(configData data1, configData data2) 
{ 
    if (data1.SettingName.Equals(data2.SettingName)) 
    { 
     string m1 = data1.SettingMachine; 
     string m2 = data2.SettingMachine; 
     if (m1 == null) 
      m1 = string.Empty; 
     if (m2 == null) 
      m2 = string.Empty; 
     return String.CompareOrdinal(m1, m2); ; 
    } 

    return String.CompareOrdinal(data1.SettingName, data2.SettingName); 
}); 

Answer 1

ReSharper le preocupa que si tuviera que ejecutar su código en un sistema de que usaba una codificación de caracteres predeterminada diferente, la comparación podría fallar. CompareOrdinal es invariante de cultura, eliminando así el problema.

Answer 2

Mi conjetura es que ReSharper está tratando de prevenir nulas referencias.

¿Qué pasaría si data1 o data2 es nulo al usar el primer fragmento de código? Compare esto con el segundo fragmento de código.

Es un poco más seguro de esta manera.

EDITAR

En ambos fragmentos, la primera if fracasarían, pero me refiero a las partes Compare, por supuesto :)