Crash when opening TableWorkspace

dimaaan / pgEdit

PostgreSQL GUI for developers

MIT License

0 stars 0 forks source link

Crash when opening TableWorkspace #40

Open dimaaan opened 9 years ago

dimaaan commented 9 years ago

Investigating

Crashing on databinding with Equin.ApplicationFramework.BindingListView.
Looks like this is mono bug https://bugzilla.xamarin.com/show_bug.cgi?id=15180
Sample application @waynebloss -> BindingListView -> DemoCS has the same problem: On .NET AggregateBindingListView.GetItemProperties return collection of 3 items, which is right, unlike mono 3.8 returns collection of 70 items

dimaaan commented 9 years ago

Bug localized. System.Windows.Forms.ListBindingHelper.GetListItemProperties(typeof(System.Drawing.Point), null) returns collection of 2 items in .NET and collection of 72 items on mono.

dimaaan commented 9 years ago

Copy paste of ListBindingHelper & SecurityUtils from .NET sources solve localized bug! But sample app won't run anyway because mono's CurrencyManager use internal version ListBindingHelper Investigate use of fixed version. Is there any way to swap System.Windows.Forms.ListBindingHelper to Equin.ApplicationFramework.ListBindingHelper.

waynebloss commented 9 years ago

Maybe Mono will accept a patch. If not, you can always fork their implementation of System.Windows.Forms and use that.

dimaaan commented 9 years ago

@waynebloss Well, my path is actually a copypaste from .net framework source. I'm not sure it legal to send such path to mono team. Anyway i'm going to ask about it on winforms mailing list.

dimaaan commented 9 years ago

mono Contributing page says Microsoft .NET source code: If you have looked at Microsoft’s proprietary implementation of .NET or their shared source code (which is also proprietary), you will not be able to contribute to Mono. If you would like to contribute in this area, be careful to not look into them in Visual Studio debugging experience.

But i already take a look at proprietary implementation. What have i done?! :)

So, i need to reimplement this class, but my implementation should be differ in some ways.