Page MenuHomePhabricator

Addresses hang issue seen in T28 by freetrader -- properly register Config type with Qt so it may be used in signals/slots
ClosedPublic

Authored by CCulianu on Jul 3 2017, 11:31.

Details

Summary

T28 issue was that passing Config object around confused Qt's type system so initialize() slot was never invoked. This fixes it to work.

Test Plan

build with Qt and run src/qt/bitcoin-qt. If it gets past splash screen without hanging, it works.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Owners added a reviewer: Restricted Owners Package.Jul 3 2017, 11:31
Herald added 1 blocking reviewer(s): Restricted Project. · View Herald TranscriptJul 3 2017, 11:31

We should just trash the whole qt interface instead of trying to fix it.

I've tested this using a build with Qt4 and it works on my system.
I'll try building on older Ubuntu to see if its ok there too.

I like the good explanatory comments in this diff.

Ok sounds good guys. Let me know if I should land it.

@deadalnix -- understood about trashing UI ... this is here in case freetrader wants to keep these changes until we *DO* trash the UI.. am ok with whatever.

Travis tests OK (ignoring the well known Wine issue)

I'm accepting this for now. It might be a shortlived patch while the GUI is removed in the near future.

This revision is now accepted and ready to land.Jul 4 2017, 18:26
This revision was automatically updated to reflect the committed changes.