1 2016-03-14T00:16:04  *** PRab has quit IRC
  2 2016-03-14T00:24:10  *** Guyver2 has quit IRC
  3 2016-03-14T00:46:38  *** Chris_Stewart_5 has joined #bitcoin-core-dev
  4 2016-03-14T01:14:06  *** Chris_Stewart_5 has quit IRC
  5 2016-03-14T01:30:00  *** dcousens has joined #bitcoin-core-dev
  6 2016-03-14T01:37:15  *** justanotheruser has quit IRC
  7 2016-03-14T01:38:11  *** justanotheruser has joined #bitcoin-core-dev
  8 2016-03-14T01:40:13  <dcousens> wc
  9 2016-03-14T01:40:15  *** dcousens has left #bitcoin-core-dev
 10 2016-03-14T01:55:29  *** fengling has joined #bitcoin-core-dev
 11 2016-03-14T02:00:59  *** Ylbam has quit IRC
 12 2016-03-14T02:02:56  *** paveljanik has quit IRC
 13 2016-03-14T02:03:06  *** pavel_ has joined #bitcoin-core-dev
 14 2016-03-14T02:18:04  *** PRab has joined #bitcoin-core-dev
 15 2016-03-14T02:40:01  *** belcher has quit IRC
 16 2016-03-14T02:59:33  *** fengling has quit IRC
 17 2016-03-14T03:07:21  *** fengling has joined #bitcoin-core-dev
 18 2016-03-14T03:30:35  *** anchow101 has quit IRC
 19 2016-03-14T03:38:53  *** xiangfu has joined #bitcoin-core-dev
 20 2016-03-14T03:41:08  *** jl2012 has joined #bitcoin-core-dev
 21 2016-03-14T03:44:13  *** fengling has quit IRC
 22 2016-03-14T03:51:11  *** fengling has joined #bitcoin-core-dev
 23 2016-03-14T03:52:32  *** Giszmo has quit IRC
 24 2016-03-14T04:18:56  *** Guest35513 is now known as [_smitty]
 25 2016-03-14T04:47:57  *** xiangfu_ has joined #bitcoin-core-dev
 26 2016-03-14T05:22:00  *** [_smitty] has quit IRC
 27 2016-03-14T05:59:37  *** pavel_ has quit IRC
 28 2016-03-14T06:00:08  *** dermoth has quit IRC
 29 2016-03-14T06:01:00  *** dermoth has joined #bitcoin-core-dev
 30 2016-03-14T06:21:23  *** jtimon has quit IRC
 31 2016-03-14T06:38:26  *** AaronvanW has joined #bitcoin-core-dev
 32 2016-03-14T06:51:04  *** sharkattack1999 has joined #bitcoin-core-dev
 33 2016-03-14T06:53:02  <sharkattack1999> is virwox safe
 34 2016-03-14T07:05:22  *** mr_burdell_ has joined #bitcoin-core-dev
 35 2016-03-14T07:06:12  *** lahwran has joined #bitcoin-core-dev
 36 2016-03-14T07:07:16  *** pigeons_ has joined #bitcoin-core-dev
 37 2016-03-14T07:07:16  *** heath__ has joined #bitcoin-core-dev
 38 2016-03-14T07:07:19  *** devrando1 has joined #bitcoin-core-dev
 39 2016-03-14T07:13:46  *** sharkattack1999 has quit IRC
 40 2016-03-14T07:14:35  *** davec has quit IRC
 41 2016-03-14T07:15:28  *** dermoth has quit IRC
 42 2016-03-14T07:15:30  *** mr_burdell has quit IRC
 43 2016-03-14T07:15:30  *** phantomcircuit has quit IRC
 44 2016-03-14T07:15:32  *** lauren has quit IRC
 45 2016-03-14T07:15:34  *** heath has quit IRC
 46 2016-03-14T07:15:36  *** pigeons has quit IRC
 47 2016-03-14T07:15:36  *** devrandom has quit IRC
 48 2016-03-14T07:15:38  *** gijensen has quit IRC
 49 2016-03-14T07:15:43  *** mr_burdell_ is now known as mr_burdell
 50 2016-03-14T07:16:09  *** davec has joined #bitcoin-core-dev
 51 2016-03-14T07:16:12  *** gijensen has joined #bitcoin-core-dev
 52 2016-03-14T07:16:12  *** dermoth has joined #bitcoin-core-dev
 53 2016-03-14T07:16:15  *** phantomcircuit has joined #bitcoin-core-dev
 54 2016-03-14T07:16:16  *** mr_burdell is now known as Guest67852
 55 2016-03-14T07:39:23  *** Thireus has quit IRC
 56 2016-03-14T07:56:06  *** AaronvanW has quit IRC
 57 2016-03-14T07:56:49  *** BashCo_ has quit IRC
 58 2016-03-14T08:07:59  *** Ylbam has joined #bitcoin-core-dev
 59 2016-03-14T08:14:33  *** Guyver2 has joined #bitcoin-core-dev
 60 2016-03-14T08:14:54  *** Thireus has joined #bitcoin-core-dev
 61 2016-03-14T08:16:21  *** Don_John has quit IRC
 62 2016-03-14T08:20:49  <GitHub69> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/0fa88ef784dd...3798e5de334c
 63 2016-03-14T08:20:50  <GitHub69> bitcoin/master ce7413f Luv Khemani: Add autocomplete to bitcoin-qt's console window....
 64 2016-03-14T08:20:50  <GitHub69> bitcoin/master 3798e5d Jonas Schnelli: Merge #7613: Add autocomplete to bitcoin-qt's console window....
 65 2016-03-14T08:20:59  <GitHub94> [bitcoin] jonasschnelli closed pull request #7613: Add autocomplete to bitcoin-qt's console window. (master...autocomplete) https://github.com/bitcoin/bitcoin/pull/7613
 66 2016-03-14T08:25:05  <GitHub88> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/3798e5de334c...b24266c20748
 67 2016-03-14T08:25:06  <GitHub88> bitcoin/master 21e45a0 Andrew C: Fix history deletion bug after font change...
 68 2016-03-14T08:25:07  <GitHub88> bitcoin/master b24266c Jonas Schnelli: Merge #7668: Fix history deletion bug after font size change...
 69 2016-03-14T08:25:10  <GitHub180> [bitcoin] jonasschnelli closed pull request #7668: Fix history deletion bug after font size change (master...rpcconsole-hist-fix) https://github.com/bitcoin/bitcoin/pull/7668
 70 2016-03-14T08:27:41  *** BashCo has joined #bitcoin-core-dev
 71 2016-03-14T08:28:53  *** paveljanik has joined #bitcoin-core-dev
 72 2016-03-14T08:28:53  *** paveljanik has joined #bitcoin-core-dev
 73 2016-03-14T08:31:58  *** BashCo has quit IRC
 74 2016-03-14T08:40:20  *** BashCo has joined #bitcoin-core-dev
 75 2016-03-14T09:12:47  <jonasschnelli> Hmm... getting a new warning:
 76 2016-03-14T09:12:47  <jonasschnelli> torcontrol.cpp:365:24: warning: private field 'base' is not used [-Wunused-private-field]
 77 2016-03-14T09:13:26  *** jannes has joined #bitcoin-core-dev
 78 2016-03-14T09:13:40  <jonasschnelli> But seems to be used in TorController::TorController constructor... strange..
 79 2016-03-14T09:16:06  <paveljanik> there is a global base
 80 2016-03-14T09:16:49  <paveljanik> it can be removed from TorController::TorController and commented out around l. 365
 81 2016-03-14T09:17:19  <paveljanik> I do not know if the global should be prefer or the private one there.
 82 2016-03-14T09:17:45  <paveljanik> let's wait for wumpus
 83 2016-03-14T09:29:08  <wumpus> we should probably rename the global one
 84 2016-03-14T09:29:20  *** p15x has joined #bitcoin-core-dev
 85 2016-03-14T09:29:27  <wumpus> (I'd like to go without it completely, but I don't think that's possible)
 86 2016-03-14T09:30:19  <wumpus> it would work if our initialization sequence was structured differently, and StartTorControl would return an instance instead, but with the global StartTorControl/InterruptTorControl/StopTorControl flow, some state needs to be global
 87 2016-03-14T09:30:29  <wumpus> it shouldn't be called the same as an instance variable though
 88 2016-03-14T09:31:33  *** gevs has joined #bitcoin-core-dev
 89 2016-03-14T09:39:13  *** Squidicuz has quit IRC
 90 2016-03-14T09:41:07  <wumpus> e.g. like this https://github.com/laanwj/bitcoin/commit/0a48b56bdc60f9dafaea7a7b2a770eb819dbfe55
 91 2016-03-14T09:42:26  <jonasschnelli> ACK https://github.com/laanwj/bitcoin/commit/0a48b56bdc60f9dafaea7a7b2a770eb819dbfe55
 92 2016-03-14T09:43:54  <GitHub154> [bitcoin] laanwj opened pull request #7676: torcontrol: rename global base, avoid overlap with field (master...2016_03_torcontrol_rename_base) https://github.com/bitcoin/bitcoin/pull/7676
 93 2016-03-14T09:44:54  <wumpus> eventually we could want to move toward having one libevent base, instead of having separate ones for RPC and torcontrol, but best to do this after cfields_' net refactor
 94 2016-03-14T09:45:13  <wumpus> (which will introduce a third one for the P2P)
 95 2016-03-14T09:48:00  <wumpus> so, it looks like 0.12 finally broke windows XP compatibility (https://github.com/bitcoin/bitcoin/issues/7639, https://github.com/bitcoin/bitcoin/issues/7674)
 96 2016-03-14T09:48:40  <wumpus> I'd almost say 'congratulations', if it wasn't that I'm afraid that it's some kind of race that may be hiding on other platforms as well
 97 2016-03-14T09:49:32  <wumpus> on the other hand this could be simply the new mingw (from ubuntu 14.04 i.s.o 12.04)
 98 2016-03-14T09:52:29  *** AaronvanW has joined #bitcoin-core-dev
 99 2016-03-14T09:52:29  *** AaronvanW has quit IRC
100 2016-03-14T09:52:29  *** AaronvanW has joined #bitcoin-core-dev
101 2016-03-14T10:12:38  <NicolasDorier> I would appreciate some feedback on https://github.com/bitcoin/bitcoin/issues/7677
102 2016-03-14T10:18:04  *** xiangfu has quit IRC
103 2016-03-14T10:19:45  *** xiangfu_ has quit IRC
104 2016-03-14T10:21:25  <GitHub107> [bitcoin] laanwj closed pull request #7676: torcontrol: rename global base, avoid overlap with field (master...2016_03_torcontrol_rename_base) https://github.com/bitcoin/bitcoin/pull/7676
105 2016-03-14T10:27:29  <GitHub42> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/b24266c20748...56d2c4e0fee3
106 2016-03-14T10:27:30  <GitHub42> bitcoin/master 393b22e Mustafa: Add a source file for unit test utils.
107 2016-03-14T10:27:30  <GitHub42> bitcoin/master 2fdaa25 Mustafa: Move GetTempPath() to testutil.
108 2016-03-14T10:27:31  <GitHub42> bitcoin/master 56d2c4e Wladimir J. van der Laan: Merge #7667: Move GetTempPath() to testutil...
109 2016-03-14T10:27:34  <GitHub100> [bitcoin] laanwj closed pull request #7667: Move GetTempPath() to testutil (master...patch-gettemppath) https://github.com/bitcoin/bitcoin/pull/7667
110 2016-03-14T10:28:48  *** laurentmt has joined #bitcoin-core-dev
111 2016-03-14T10:29:37  *** p15x has quit IRC
112 2016-03-14T10:30:26  <GitHub97> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/56d2c4e0fee3...f1ca8915bb22
113 2016-03-14T10:30:27  <GitHub97> bitcoin/master b0ff857 Wladimir J. van der Laan: test: Move non-generated script_invalid test to the correct place...
114 2016-03-14T10:30:27  <GitHub97> bitcoin/master 2317ad7 Wladimir J. van der Laan: test: Re-introduce JSON pretty printing in test builder
115 2016-03-14T10:30:28  <GitHub97> bitcoin/master 0ecb340 Wladimir J. van der Laan: test: Script_error checking in script_invalid tests...
116 2016-03-14T10:30:31  <GitHub32> [bitcoin] laanwj closed pull request #7517: test: script_error checking in script_invalid tests (master...2016_02_test_script_errors) https://github.com/bitcoin/bitcoin/pull/7517
117 2016-03-14T10:32:21  *** p15x has joined #bitcoin-core-dev
118 2016-03-14T10:32:40  *** fengling has quit IRC
119 2016-03-14T10:34:11  <wumpus> paveljanik: I don't like that warning - it's correct, after #7637 we don't need to keep `base` around at all in TorController right now because we only use it in the constructor.  But I think it's good practice to do so anyway, in case it's needed again in later changes...
120 2016-03-14T10:35:09  <GitHub160> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/f1ca8915bb22...5b3b5a7d711c
121 2016-03-14T10:35:10  <GitHub160> bitcoin/master fc7c60d Jonas Schnelli: [Wallet] move "load wallet phase" to CWallet
122 2016-03-14T10:35:10  <GitHub160> bitcoin/master 15e6e13 Jonas Schnelli: [Wallet] optimize return value of InitLoadWallet()
123 2016-03-14T10:35:11  <GitHub160> bitcoin/master 5b3b5a7 Wladimir J. van der Laan: Merge #7577: [Wallet] move "load wallet phase" to CWallet...
124 2016-03-14T10:35:17  <GitHub41> [bitcoin] laanwj closed pull request #7577: [Wallet] move "load wallet phase" to CWallet (master...2016/02/wallet_ref2) https://github.com/bitcoin/bitcoin/pull/7577
125 2016-03-14T10:35:59  <GitHub130> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/5b3b5a7d711c...11c769966ad7
126 2016-03-14T10:35:59  <GitHub130> bitcoin/master 8d1de43 Leviathn: Remove internal miner...
127 2016-03-14T10:36:00  <GitHub130> bitcoin/master 11c7699 Wladimir J. van der Laan: Merge #7507: Remove internal miner...
128 2016-03-14T10:36:02  <GitHub30> [bitcoin] laanwj closed pull request #7507: Remove internal miner (master...master) https://github.com/bitcoin/bitcoin/pull/7507
129 2016-03-14T10:36:37  <GitHub155> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/11c769966ad7...c87f51e55bc3
130 2016-03-14T10:36:37  <GitHub155> bitcoin/master 8a253b3 Pieter Wuille: Make the generate RPC call function for non-regtest
131 2016-03-14T10:36:38  <GitHub155> bitcoin/master c87f51e Wladimir J. van der Laan: Merge #7663: Make the generate RPC call function for non-regtest...
132 2016-03-14T10:36:47  <GitHub3> [bitcoin] laanwj closed pull request #7663: Make the generate RPC call function for non-regtest (master...generatenonreg) https://github.com/bitcoin/bitcoin/pull/7663
133 2016-03-14T10:37:01  <paveljanik> wumpus, 8) When it is needed in later changes, it can be resurrected from git history 8)
134 2016-03-14T10:37:22  <wumpus> paveljanik: of course, but I think the warning is over-eager
135 2016-03-14T10:38:29  <paveljanik> every warning is disturbing, hides probles etc. But I know you have a different opinion :-)
136 2016-03-14T10:39:08  <paveljanik> and I respect that.
137 2016-03-14T10:39:17  <wumpus> sure, they're annoying, but in some cases it makes more sense to disable the warning than change the code
138 2016-03-14T10:40:03  <paveljanik> I agree with that - in some cases. Like known compiler bug etc...
139 2016-03-14T10:40:06  <wumpus> it's not up to compiler authors to decide about our code style
140 2016-03-14T10:40:45  <wumpus> looks like they're making the compiler behave like a nanny with too much time on their hands
141 2016-03-14T10:41:00  <paveljanik> I agree with that completely ;-)
142 2016-03-14T10:42:01  <wumpus> all time spent on 'fixing' the code to be as acceptable as possible to the compiler is not spent on facing user-facing issues, ok and that concludes my rant on this
143 2016-03-14T10:44:40  *** paveljanik has quit IRC
144 2016-03-14T11:10:25  <GitHub102> [bitcoin] laanwj closed pull request #7655: [qt] Add .clang-format (master...Mf1603-qt-clang-format-0.12) https://github.com/bitcoin/bitcoin/pull/7655
145 2016-03-14T11:23:08  <GitHub7> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c87f51e55bc3...c9a12657545f
146 2016-03-14T11:23:08  <GitHub7> bitcoin/master a6ee0ca Pavel Vasin: use cached block hash in blockToJSON()
147 2016-03-14T11:23:09  <GitHub7> bitcoin/master c9a1265 Wladimir J. van der Laan: Merge #7670: use cached block hash in blockToJSON()...
148 2016-03-14T11:23:18  <GitHub70> [bitcoin] laanwj closed pull request #7670: use cached block hash in blockToJSON() (master...patch) https://github.com/bitcoin/bitcoin/pull/7670
149 2016-03-14T11:24:43  <GitHub199> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c9a12657545f...c15eb2824184
150 2016-03-14T11:24:43  <GitHub199> bitcoin/master ce41cf0 BtcDrak: Add curl to Gitian setup instrustions...
151 2016-03-14T11:24:44  <GitHub199> bitcoin/master c15eb28 Wladimir J. van der Laan: Merge #7658: Add curl to Gitian setup instructions...
152 2016-03-14T11:24:48  <GitHub192> [bitcoin] laanwj closed pull request #7658: Add curl to Gitian setup instructions (master...curl) https://github.com/bitcoin/bitcoin/pull/7658
153 2016-03-14T11:26:54  *** xabbix_ has joined #bitcoin-core-dev
154 2016-03-14T11:26:55  *** MarcoFalke has joined #bitcoin-core-dev
155 2016-03-14T11:28:06  *** xabbix has quit IRC
156 2016-03-14T11:55:05  <GitHub32> [bitcoin] laanwj opened pull request #7680: qt: Remove reflection from `about` icon (master...2016_03_image_optimize) https://github.com/bitcoin/bitcoin/pull/7680
157 2016-03-14T11:59:25  *** laurentmt has quit IRC
158 2016-03-14T12:02:30  *** Guest67852 is now known as mr_burdell
159 2016-03-14T12:02:44  *** mr_burdell has quit IRC
160 2016-03-14T12:02:44  *** mr_burdell has joined #bitcoin-core-dev
161 2016-03-14T12:27:30  *** wasi has joined #bitcoin-core-dev
162 2016-03-14T12:28:02  *** BashCo_ has joined #bitcoin-core-dev
163 2016-03-14T12:32:02  *** BashCo has quit IRC
164 2016-03-14T12:43:26  <GitHub195> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c15eb2824184...41d2dfa9e73b
165 2016-03-14T12:43:26  <GitHub195> bitcoin/master 322a7a2 Wladimir J. van der Laan: qt: Remove reflection from `about` icon...
166 2016-03-14T12:43:27  <GitHub195> bitcoin/master 41d2dfa Wladimir J. van der Laan: Merge #7680: qt: Remove reflection from `about` icon...
167 2016-03-14T12:43:33  <GitHub2> [bitcoin] laanwj closed pull request #7680: qt: Remove reflection from `about` icon (master...2016_03_image_optimize) https://github.com/bitcoin/bitcoin/pull/7680
168 2016-03-14T12:50:15  *** jyap has quit IRC
169 2016-03-14T12:52:06  *** jyap has joined #bitcoin-core-dev
170 2016-03-14T12:52:06  *** jyap has joined #bitcoin-core-dev
171 2016-03-14T12:56:44  <GitHub179> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/41d2dfa9e73b...0735c0ca7c22
172 2016-03-14T12:56:45  <GitHub179> bitcoin/master fa19b18 MarcoFalke: [wallet] Move hardcoded file name out of log messages
173 2016-03-14T12:56:46  <GitHub179> bitcoin/master 0735c0c Wladimir J. van der Laan: Merge #7608: [wallet] Move hardcoded file name out of log messages...
174 2016-03-14T12:56:53  <GitHub194> [bitcoin] laanwj closed pull request #7608: [wallet] Move hardcoded file name out of log messages (master...Mf1602-walletFileName) https://github.com/bitcoin/bitcoin/pull/7608
175 2016-03-14T13:00:26  *** laurentmt has joined #bitcoin-core-dev
176 2016-03-14T13:04:02  *** Chris_Stewart_5 has joined #bitcoin-core-dev
177 2016-03-14T13:13:01  *** jtimon has joined #bitcoin-core-dev
178 2016-03-14T13:35:04  *** Chris_Stewart_5 has quit IRC
179 2016-03-14T13:50:34  *** Guyver2 has quit IRC
180 2016-03-14T14:06:35  <GitHub98> [bitcoin] MarcoFalke opened pull request #7682: [0.11.3] Backports (0.11...Mf1603-011wallet) https://github.com/bitcoin/bitcoin/pull/7682
181 2016-03-14T14:13:17  <MarcoFalke> wumpus, what is the plan for 0.10.4 and 0.11.3?
182 2016-03-14T14:13:25  <wumpus> no plan at this moment
183 2016-03-14T14:14:23  <wumpus> another 0.10.x would require a critical reason, we could do a 0.11.x if there's good reason
184 2016-03-14T14:17:22  <btcdrak> the next maint release should be the CSV softfork imo
185 2016-03-14T14:17:53  *** paveljanik has joined #bitcoin-core-dev
186 2016-03-14T14:20:27  <wumpus> sure, well, to be honest I don't really keep track of 0.11 backports, may make sense to appoint another release manager for backport releases
187 2016-03-14T14:20:40  <wumpus> (e.g. as linux has)
188 2016-03-14T14:20:49  <MarcoFalke> 0.11 currently has my "[dbwrapper] Detect obfuscation" patch and the wallet issues
189 2016-03-14T14:21:11  <MarcoFalke> Though, it also includes the "paytxfee changed behavior without warning"
190 2016-03-14T14:21:37  <wumpus> possibly Luke-Jr, though maybe he is a bit too trigger-happy with backporting
191 2016-03-14T14:21:54  <wumpus> the list in https://github.com/bitcoin/bitcoin/pull/7047 is pretty scary :)
192 2016-03-14T14:22:22  *** xabbix_ has quit IRC
193 2016-03-14T14:22:37  <MarcoFalke> Jup, preferrably only changes that the end user can see should be backported.
194 2016-03-14T14:22:47  <wumpus> the changed fee behavior is backported to 0.11?
195 2016-03-14T14:23:17  *** wasi has quit IRC
196 2016-03-14T14:23:18  *** xabbix_ has joined #bitcoin-core-dev
197 2016-03-14T14:23:32  <wumpus> ah yeah: * 7d0a05f - Default fPayAtLeastCustomFee to false
198 2016-03-14T14:23:44  <wumpus> going to revert that one, I don't want to do that on th 0.11 branch
199 2016-03-14T14:28:22  <GitHub35> [bitcoin] laanwj pushed 1 new commit to 0.11: https://github.com/bitcoin/bitcoin/commit/b6f59048d33a360246e000fc45daf26d08a693de
200 2016-03-14T14:28:22  <GitHub35> bitcoin/0.11 b6f5904 Wladimir J. van der Laan: Revert "Default fPayAtLeastCustomFee to false"...
201 2016-03-14T14:29:54  <MarcoFalke> Don't forget to revert 5f09cda0bf4cddc7d33989717a97ae53e85a7a00 as well.
202 2016-03-14T14:30:27  <MarcoFalke> The unit tests will fail now because the wallet pays too much.
203 2016-03-14T14:30:40  <wumpus> good point
204 2016-03-14T14:31:03  <GitHub97> [bitcoin] laanwj pushed 1 new commit to 0.11: https://github.com/bitcoin/bitcoin/commit/d49f72420b86cff446961f007dce050b84594373
205 2016-03-14T14:31:03  <GitHub97> bitcoin/0.11 d49f724 Wladimir J. van der Laan: Revert "[wallet] Add rpc tests to verify fee calculations"...
206 2016-03-14T14:41:49  *** treehug88 has joined #bitcoin-core-dev
207 2016-03-14T14:51:29  *** Giszmo has joined #bitcoin-core-dev
208 2016-03-14T14:57:38  *** Chris_Stewart_5 has joined #bitcoin-core-dev
209 2016-03-14T14:58:30  *** Chris_Stewart_5 has joined #bitcoin-core-dev
210 2016-03-14T15:15:43  <jonasschnelli> wumpus: this fixes the torcontrol warning for me: https://github.com/bitcoin/bitcoin/compare/master...jonasschnelli:2016/03/torctrl?expand=1
211 2016-03-14T15:16:03  <jonasschnelli> But the current constructor should be "legal" -> base(base)
212 2016-03-14T15:16:16  <jonasschnelli> Not sure why it does fix the clang warning...
213 2016-03-14T15:20:49  *** laurentmt has quit IRC
214 2016-03-14T15:28:25  *** laurentmt has joined #bitcoin-core-dev
215 2016-03-14T15:34:16  <GitHub147> [bitcoin] jonasschnelli opened pull request #7683: Fix torcontrol.cpp unused private field warning (master...2016/03/torctrl) https://github.com/bitcoin/bitcoin/pull/7683
216 2016-03-14T15:34:41  <jonasschnelli> paveljanik maybe verify https://github.com/bitcoin/bitcoin/pull/7683?
217 2016-03-14T15:35:11  <paveljanik> will do
218 2016-03-14T15:35:42  <paveljanik> hmm
219 2016-03-14T15:35:57  <paveljanik> I'd like to see a mix of your and Wladimir's solution ;-)
220 2016-03-14T15:36:20  <paveljanik> it is not clear now, what is the concept behind two variables, named the same.
221 2016-03-14T15:36:58  <paveljanik> it is not clear what "base" is used from the first review of the source.
222 2016-03-14T15:37:14  <paveljanik> but I'll confirm if it fixes the warning
223 2016-03-14T15:37:53  *** Thireus has quit IRC
224 2016-03-14T15:43:45  *** wasi has joined #bitcoin-core-dev
225 2016-03-14T15:46:40  <GitHub148> [bitcoin] MarcoFalke opened pull request #7684: [qa] Extend tests (master...Mf1603-qaCleanup1) https://github.com/bitcoin/bitcoin/pull/7684
226 2016-03-14T15:48:51  <jonasschnelli> paveljanik: I agree the var names should be different... but no strong opinion. The struct event_base *base; at L662 should not interfere the private struct event_base* base; of TorController (not even shadow).
227 2016-03-14T15:50:09  <paveljanik> jonasschnelli, and this is what your and wumpus 's change does :-)
228 2016-03-14T15:50:19  <paveljanik> but anyway, warning gone with your PR
229 2016-03-14T15:50:24  <jonasschnelli> +1
230 2016-03-14T15:50:38  <jonasschnelli> paveljanik: thanks for testing...
231 2016-03-14T15:51:09  <paveljanik> it was done almost without me, automagically 8)
232 2016-03-14T15:52:58  *** Guyver2 has joined #bitcoin-core-dev
233 2016-03-14T15:58:58  *** Thireus has joined #bitcoin-core-dev
234 2016-03-14T16:14:01  *** BashCo_ has quit IRC
235 2016-03-14T16:16:36  *** dermoth has quit IRC
236 2016-03-14T16:20:54  *** rubensayshi has joined #bitcoin-core-dev
237 2016-03-14T16:23:57  *** laurentmt has quit IRC
238 2016-03-14T16:31:00  *** treehug88 has quit IRC
239 2016-03-14T16:31:11  *** treehug88 has joined #bitcoin-core-dev
240 2016-03-14T16:36:36  *** BashCo has joined #bitcoin-core-dev
241 2016-03-14T16:50:38  *** treehug88 has quit IRC
242 2016-03-14T16:53:04  <GitHub73> [bitcoin] jonasschnelli opened pull request #7685: [WIP] Add bloom filter usage statistics (master...2016/03/bf_stats) https://github.com/bitcoin/bitcoin/pull/7685
243 2016-03-14T17:03:37  *** treehug88 has joined #bitcoin-core-dev
244 2016-03-14T17:10:02  *** laurentmt has joined #bitcoin-core-dev
245 2016-03-14T17:15:10  *** dermoth has joined #bitcoin-core-dev
246 2016-03-14T17:28:34  *** dermoth has quit IRC
247 2016-03-14T17:38:08  <GitHub16> [bitcoin] MarcoFalke opened pull request #7686: [qt] Remove 0-fee from send dialog (master...Mf1603-qt-0-fee) https://github.com/bitcoin/bitcoin/pull/7686
248 2016-03-14T17:38:12  <MarcoFalke> spring-cleaning. Let's remove all the code!
249 2016-03-14T17:38:32  <GitHub33> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/0735c0ca7c22...48f39058315c
250 2016-03-14T17:38:33  <GitHub33> bitcoin/master 3d7e973 Jonas Schnelli: Fix torcontrol.cpp unused private field warning
251 2016-03-14T17:38:33  <GitHub33> bitcoin/master 48f3905 Wladimir J. van der Laan: Merge #7683: Fix torcontrol.cpp unused private field warning...
252 2016-03-14T17:38:40  <GitHub32> [bitcoin] laanwj closed pull request #7683: Fix torcontrol.cpp unused private field warning (master...2016/03/torctrl) https://github.com/bitcoin/bitcoin/pull/7683
253 2016-03-14T17:45:11  <morcos> btcdrak: ok, i submitted a PR to your repo for my new RPC test.  it's a handful, but i think it now tests much much more of the consensus logic.
254 2016-03-14T17:56:39  *** Tasoshi has joined #bitcoin-core-dev
255 2016-03-14T18:03:58  <GitHub177> [bitcoin] sipa opened pull request #7687: Stop treating importaddress'ed scripts as change (master...fixscriptimport) https://github.com/bitcoin/bitcoin/pull/7687
256 2016-03-14T18:07:33  *** laurentmt has quit IRC
257 2016-03-14T18:22:04  <btcdrak> morcos: thank you very much! I'll take a look after dinner.
258 2016-03-14T18:22:46  <GitHub73> [bitcoin] sipa opened pull request #7688: List solvability in listunspent output and improve help (master...helpspendsolv) https://github.com/bitcoin/bitcoin/pull/7688
259 2016-03-14T19:06:44  *** wallet42 has joined #bitcoin-core-dev
260 2016-03-14T19:16:29  *** Guyver2_ has joined #bitcoin-core-dev
261 2016-03-14T19:19:16  *** achow101 has joined #bitcoin-core-dev
262 2016-03-14T19:19:47  *** Guyver2 has quit IRC
263 2016-03-14T19:19:53  *** Guyver2_ is now known as Guyver2
264 2016-03-14T19:32:50  *** dirtynewshoes has joined #bitcoin-core-dev
265 2016-03-14T19:50:09  *** skyraider has joined #bitcoin-core-dev
266 2016-03-14T20:23:13  *** schmidty has joined #bitcoin-core-dev
267 2016-03-14T20:46:42  *** zooko has joined #bitcoin-core-dev
268 2016-03-14T20:49:55  *** Don_John has joined #bitcoin-core-dev
269 2016-03-14T20:55:52  *** d_t has quit IRC
270 2016-03-14T20:58:03  *** Don_John has joined #bitcoin-core-dev
271 2016-03-14T21:03:13  *** Don_John has joined #bitcoin-core-dev
272 2016-03-14T21:26:19  *** JackH has joined #bitcoin-core-dev
273 2016-03-14T21:44:44  *** pigeons_ is now known as pigeons
274 2016-03-14T21:50:33  *** skyraider has quit IRC
275 2016-03-14T21:55:32  *** jannes has quit IRC
276 2016-03-14T22:00:25  *** treehug88 has quit IRC
277 2016-03-14T22:03:11  *** [1]evoskuil has joined #bitcoin-core-dev
278 2016-03-14T22:05:14  *** evoskuil has quit IRC
279 2016-03-14T22:07:34  *** [1]evoskuil has quit IRC
280 2016-03-14T22:08:01  *** evoskuil has joined #bitcoin-core-dev
281 2016-03-14T22:14:01  *** zooko has quit IRC
282 2016-03-14T22:43:46  *** Guyver2 has quit IRC
283 2016-03-14T22:47:09  <GitHub87> [bitcoin] sipa opened pull request #7689: Replace OpenSSL AES with our own constant-time version (edit of #5949) (master...const_aes) https://github.com/bitcoin/bitcoin/pull/7689
284 2016-03-14T22:47:53  *** d_t has joined #bitcoin-core-dev
285 2016-03-14T22:49:38  *** d_t_ has joined #bitcoin-core-dev
286 2016-03-14T22:49:43  *** d_t has quit IRC
287 2016-03-14T23:20:51  *** d_t_ has quit IRC
288 2016-03-14T23:23:36  *** gevs has quit IRC
289 2016-03-14T23:49:58  *** Thireus has quit IRC