Merge pull request #1247 from github/fixes/culture-null

jcansdale · web-flow · commit 1dd3eebc5d5d · 2017-10-20T12:06:53.000+01:00
Visual Studio sometimes crashes after selecting Clone on Team Explorer - Connect
diff --git a/src/GitHub.VisualStudio/UI/Views/Controls/RepositoryCloneControl.xaml.cs b/src/GitHub.VisualStudio/UI/Views/Controls/RepositoryCloneControl.xaml.cs
@@ -88,8 +88,6 @@ public RepositoryGroupDescription(RepositoryCloneControl owner)
 
             public override object GroupNameFromItem(object item, int level, System.Globalization.CultureInfo culture)
             {
-                Guard.ArgumentNotNull(culture, nameof(culture));
-
                 var repo = item as IRemoteRepositoryModel;
                 var name = repo?.Owner ?? string.Empty;
                 RepositoryGroup group;

Original file line number	Diff line number	Diff line change
`@@ -88,8 +88,6 @@ public RepositoryGroupDescription(RepositoryCloneControl owner)`
`88`	`88`
`89`	`89`	`public override object GroupNameFromItem(object item, int level, System.Globalization.CultureInfo culture)`
`90`	`90`	`{`
`91`		`- Guard.ArgumentNotNull(culture, nameof(culture));`
`92`		`-`
`93`	`91`	`var repo = item as IRemoteRepositoryModel;`
`94`	`92`	`var name = repo?.Owner ?? string.Empty;`
`95`	`93`	`RepositoryGroup group;`