- move all QApplication includes to top of included Qt headers
- undef our loop macro where it would cause compilation errors otherwise
Bitcoin-Qt: fix QApplication includes to match our include defaults #2847
pull Diapolo wants to merge 1 commits into bitcoin:master from Diapolo:fix_QApplication_include changing 7 files +13 −12-
Diapolo commented at 6:55 AM on July 23, 2013: none
-
84ef729aa1
Bitcoin-Qt: fix QApplication includes to match our include defaults
- move all QApplication includes to top of included Qt headers - undef our loop macro where it would cause compilation errors otherwise
-
BitcoinPullTester commented at 7:24 AM on July 23, 2013: none
Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/84ef729aa1de762229c8a5c800cddff1a014fcca for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
- laanwj referenced this in commit 2b894bceac on Jul 26, 2013
- laanwj merged this on Jul 26, 2013
- laanwj closed this on Jul 26, 2013
- IntegralTeam referenced this in commit 27b2cd2cca on Jun 4, 2019
- DrahtBot locked this on Sep 8, 2021
Contributors