12017-04-09T00:14:56 *** alpalp has joined #bitcoin-core-dev
22017-04-09T00:14:56 *** alpalp has joined #bitcoin-core-dev
32017-04-09T00:19:27 *** fremenWalking has joined #bitcoin-core-dev
42017-04-09T00:30:05 *** Samdney has quit IRC
52017-04-09T00:43:03 *** jtimon has joined #bitcoin-core-dev
62017-04-09T00:59:08 *** Char0n has joined #bitcoin-core-dev
72017-04-09T01:36:44 *** d_t has quit IRC
82017-04-09T01:44:42 *** juscamarena has joined #bitcoin-core-dev
92017-04-09T02:02:12 *** laurentmt has joined #bitcoin-core-dev
102017-04-09T02:04:18 *** laurentmt has quit IRC
112017-04-09T02:24:09 *** Ylbam has quit IRC
122017-04-09T02:40:32 *** alpalp has quit IRC
132017-04-09T02:50:24 *** vicenteH has quit IRC
142017-04-09T03:05:40 *** jtimon has quit IRC
152017-04-09T03:15:09 *** d_t has joined #bitcoin-core-dev
162017-04-09T03:41:59 *** To7 has quit IRC
172017-04-09T04:09:52 *** cryptapus has joined #bitcoin-core-dev
182017-04-09T04:10:11 *** cryptapus is now known as cryptapus_afk
192017-04-09T04:26:24 *** mol has joined #bitcoin-core-dev
202017-04-09T04:29:28 *** molz_ has quit IRC
212017-04-09T04:49:23 *** n1ce has quit IRC
222017-04-09T04:49:36 *** justanotheruser has joined #bitcoin-core-dev
232017-04-09T04:51:04 *** n1ce has joined #bitcoin-core-dev
242017-04-09T04:52:01 *** cryptapus_afk has quit IRC
252017-04-09T04:56:11 *** n1ce has quit IRC
262017-04-09T04:56:13 *** cryptapus has joined #bitcoin-core-dev
272017-04-09T04:56:14 *** cryptapus has joined #bitcoin-core-dev
282017-04-09T04:56:38 *** cryptapus is now known as cryptapus_afk
292017-04-09T05:24:52 *** btcdrak has quit IRC
302017-04-09T06:28:26 *** juscamarena has quit IRC
312017-04-09T06:28:49 *** juscamarena has joined #bitcoin-core-dev
322017-04-09T06:31:03 *** juscamarena has joined #bitcoin-core-dev
332017-04-09T06:37:04 *** juscamarena has quit IRC
342017-04-09T06:37:20 *** dgenr8 has quit IRC
352017-04-09T06:37:50 *** juscamarena has joined #bitcoin-core-dev
362017-04-09T06:37:55 *** dgenr8 has joined #bitcoin-core-dev
372017-04-09T06:39:39 *** CubicEarth has joined #bitcoin-core-dev
382017-04-09T06:52:05 *** CubicEarth has quit IRC
392017-04-09T06:53:03 *** juscamarena has quit IRC
402017-04-09T06:56:24 *** juscamarena has joined #bitcoin-core-dev
412017-04-09T06:58:24 *** juscamarena has quit IRC
422017-04-09T06:58:57 *** juscamarena has joined #bitcoin-core-dev
432017-04-09T06:59:12 *** btcdrak has joined #bitcoin-core-dev
442017-04-09T07:03:27 *** juscamarena has quit IRC
452017-04-09T07:04:05 *** juscamarena has joined #bitcoin-core-dev
462017-04-09T07:05:50 *** Guyver2 has joined #bitcoin-core-dev
472017-04-09T07:07:12 *** juscamarena has quit IRC
482017-04-09T07:08:25 *** juscamarena has joined #bitcoin-core-dev
492017-04-09T07:46:15 <wumpus> luke-jr: ok, feel free to change that
502017-04-09T07:47:58 *** Ylbam has joined #bitcoin-core-dev
512017-04-09T07:48:29 *** CubicEarth has joined #bitcoin-core-dev
522017-04-09T07:51:30 *** Guyver2 has quit IRC
532017-04-09T08:06:31 *** CubicEarth has quit IRC
542017-04-09T08:11:23 *** d_t has quit IRC
552017-04-09T08:26:44 *** juscamarena has quit IRC
562017-04-09T08:31:25 *** juscamarena has joined #bitcoin-core-dev
572017-04-09T08:47:10 *** juscamarena has quit IRC
582017-04-09T09:01:41 *** PaulCapestany has quit IRC
592017-04-09T09:18:14 *** PaulCapestany has joined #bitcoin-core-dev
602017-04-09T09:41:21 *** vicenteH has joined #bitcoin-core-dev
612017-04-09T10:22:20 *** AaronvanW has joined #bitcoin-core-dev
622017-04-09T10:43:41 *** RCasatta has joined #bitcoin-core-dev
632017-04-09T10:50:09 *** RCasatta has quit IRC
642017-04-09T10:50:34 *** afk11 has quit IRC
652017-04-09T10:50:35 *** ^arubi has quit IRC
662017-04-09T10:50:35 *** wasi has quit IRC
672017-04-09T10:51:50 *** jtimon has joined #bitcoin-core-dev
682017-04-09T10:56:05 *** afk11 has joined #bitcoin-core-dev
692017-04-09T10:58:13 *** arubi has joined #bitcoin-core-dev
702017-04-09T11:31:06 <stevenroose> I haven't looked into Core's code too much, but I feel like there is quite a lack of code documentation..
712017-04-09T11:31:08 <stevenroose> F.e. https://github.com/bitcoin/bitcoin/blob/f93f9b92969b4a3f77d3f1c2a30ba3bf558e0cac/src/txmempool.cpp#L759
722017-04-09T11:31:22 <stevenroose> It says "compare" and returns a bool.
732017-04-09T11:31:49 <stevenroose> Compare methods commonly return an int that is negative for <, 0 for == and positive for >
742017-04-09T11:32:05 <stevenroose> Also, it might be useful to document what it is basing the comparison on
752017-04-09T11:32:51 <stevenroose> Even thought the method name says depth and score, it doesn;t say if a higher depth and higher score mean higher ranking (which is then either true or false)
762017-04-09T11:32:58 *** alpalp has joined #bitcoin-core-dev
772017-04-09T11:32:58 *** alpalp has joined #bitcoin-core-dev
782017-04-09T11:33:01 *** JackH_ is now known as JackH
792017-04-09T11:38:09 *** Guest85725 has joined #bitcoin-core-dev
802017-04-09T11:45:00 *** Guest85725 has quit IRC
812017-04-09T12:03:19 *** Ruben has joined #bitcoin-core-dev
822017-04-09T12:03:58 *** wasi has joined #bitcoin-core-dev
832017-04-09T12:09:11 *** alpalp has quit IRC
842017-04-09T12:28:04 *** felco has quit IRC
852017-04-09T12:29:35 <sipa> stevenroose: in C++ a comparator is a function that acts like (a < b)
862017-04-09T12:35:57 <gmaxwell> s/C++/stl/
872017-04-09T12:41:53 *** Alina-malina has quit IRC
882017-04-09T12:45:17 *** jtimon has quit IRC
892017-04-09T12:46:38 *** Alina-malina has joined #bitcoin-core-dev
902017-04-09T12:49:00 *** Alina-malina has quit IRC
912017-04-09T12:49:00 *** Alina-malina has joined #bitcoin-core-dev
922017-04-09T12:58:15 *** jouke has quit IRC
932017-04-09T12:58:58 <sipa> gmaxwell: which is part of the C++ standard, but ok, other libraries can do things differently
942017-04-09T13:00:23 *** jouke has joined #bitcoin-core-dev
952017-04-09T13:06:15 *** Guyver2 has joined #bitcoin-core-dev
962017-04-09T13:26:51 *** Ruben has quit IRC
972017-04-09T13:30:43 *** Ruben has joined #bitcoin-core-dev
982017-04-09T13:47:25 <sipa> gmaxwell: i guess i should have said "in c++ it is common practice to use comparators which compute the less-than relation"
992017-04-09T13:50:13 *** d_t has joined #bitcoin-core-dev
1002017-04-09T13:58:34 <gmaxwell> it just sounded weird to say "in C++"
1012017-04-09T14:04:48 *** Samdney has joined #bitcoin-core-dev
1022017-04-09T14:16:56 *** arowser has quit IRC
1032017-04-09T14:18:10 *** Ruben has quit IRC
1042017-04-09T14:18:12 *** moctos has joined #bitcoin-core-dev
1052017-04-09T14:18:27 *** arowser has joined #bitcoin-core-dev
1062017-04-09T14:19:27 *** mol has quit IRC
1072017-04-09T14:22:49 *** moli_ has joined #bitcoin-core-dev
1082017-04-09T14:23:35 *** moli_ has joined #bitcoin-core-dev
1092017-04-09T14:43:30 *** moli_ has quit IRC
1102017-04-09T14:43:48 <bitcoin-git> [bitcoin] sipa closed pull request #8087: Introduce CBlockchain and move CheckBlockHeader (master...2016-05-22-cblockchain) https://github.com/bitcoin/bitcoin/pull/8087
1112017-04-09T14:44:04 *** moli_ has joined #bitcoin-core-dev
1122017-04-09T15:36:15 *** jtimon has joined #bitcoin-core-dev
1132017-04-09T15:55:02 *** Sosumi has joined #bitcoin-core-dev
1142017-04-09T16:01:23 <bitcoin-git> [bitcoin] sipa closed pull request #9599: [WIP] Perform validation in a separate thread to message handling. (master...ValidationThread) https://github.com/bitcoin/bitcoin/pull/9599
1152017-04-09T16:11:32 *** moctos has quit IRC
1162017-04-09T16:11:35 *** Guyver2 has quit IRC
1172017-04-09T16:23:26 *** RubenSomsen has joined #bitcoin-core-dev
1182017-04-09T16:38:35 *** Ruben has joined #bitcoin-core-dev
1192017-04-09T16:42:10 *** RubenSomsen has quit IRC
1202017-04-09T16:46:34 *** RubenSomsen has joined #bitcoin-core-dev
1212017-04-09T16:49:31 *** Ruben has quit IRC
1222017-04-09T16:55:16 *** bitcoinreminder_ has joined #bitcoin-core-dev
1232017-04-09T16:59:32 *** To7 has joined #bitcoin-core-dev
1242017-04-09T17:00:52 *** Ruben has joined #bitcoin-core-dev
1252017-04-09T17:04:31 *** RubenSomsen has quit IRC
1262017-04-09T17:10:13 *** RubenSomsen has joined #bitcoin-core-dev
1272017-04-09T17:13:10 *** Ruben has quit IRC
1282017-04-09T17:26:36 *** Ruben has joined #bitcoin-core-dev
1292017-04-09T17:28:11 *** RubenSomsen has quit IRC
1302017-04-09T17:31:19 *** aantonop has joined #bitcoin-core-dev
1312017-04-09T17:45:10 *** Ruben has quit IRC
1322017-04-09T17:46:37 *** RubenSomsen has joined #bitcoin-core-dev
1332017-04-09T17:56:06 *** Sosumi has quit IRC
1342017-04-09T18:18:51 *** aantonop has quit IRC
1352017-04-09T18:19:25 *** aantonop has joined #bitcoin-core-dev
1362017-04-09T18:22:06 *** mol has joined #bitcoin-core-dev
1372017-04-09T18:25:11 *** moli_ has quit IRC
1382017-04-09T18:29:29 *** Giszmo has joined #bitcoin-core-dev
1392017-04-09T18:30:17 *** aantonop has joined #bitcoin-core-dev
1402017-04-09T18:39:50 *** d_t has joined #bitcoin-core-dev
1412017-04-09T18:54:10 *** RubenSomsen has quit IRC
1422017-04-09T19:04:53 *** dermoth has quit IRC
1432017-04-09T19:05:00 *** dermoth_ has joined #bitcoin-core-dev
1442017-04-09T19:05:30 *** dermoth_ is now known as dermoth
1452017-04-09T19:14:10 *** Chris_Stewart_5 has quit IRC
1462017-04-09T19:14:51 *** Giszmo has quit IRC
1472017-04-09T19:19:32 *** dermoth has quit IRC
1482017-04-09T19:29:47 *** Giszmo has joined #bitcoin-core-dev
1492017-04-09T19:31:02 *** Chris_Stewart_5 has joined #bitcoin-core-dev
1502017-04-09T19:32:08 *** dermoth has joined #bitcoin-core-dev
1512017-04-09T19:36:53 *** aantonop has quit IRC
1522017-04-09T19:37:34 *** owowo has quit IRC
1532017-04-09T19:37:58 *** owowo has joined #bitcoin-core-dev
1542017-04-09T19:43:53 *** aantonop has joined #bitcoin-core-dev
1552017-04-09T19:44:52 *** aantonop has joined #bitcoin-core-dev
1562017-04-09T19:48:26 *** dacox has quit IRC
1572017-04-09T19:50:36 *** moli_ has joined #bitcoin-core-dev
1582017-04-09T19:50:51 *** mol has quit IRC
1592017-04-09T19:53:42 *** aantonop has quit IRC
1602017-04-09T19:53:52 *** aantonop has joined #bitcoin-core-dev
1612017-04-09T19:56:04 *** aantonop has joined #bitcoin-core-dev
1622017-04-09T20:20:12 <stevenroose> well, fwiw, Go's sort.Interface.Less() method does the same thing, so I figured as much. but still "Less()" is more explanatory than "operator()". but thanks, I'm making sense of the code
1632017-04-09T20:24:02 *** aantonop has quit IRC
1642017-04-09T20:24:47 *** aantonop has joined #bitcoin-core-dev
1652017-04-09T20:26:21 *** aantonop has quit IRC
1662017-04-09T20:27:25 *** aantonop has joined #bitcoin-core-dev
1672017-04-09T20:34:17 *** aantonop has quit IRC
1682017-04-09T20:34:27 *** aantonop has joined #bitcoin-core-dev
1692017-04-09T20:44:53 *** aantonop has quit IRC
1702017-04-09T20:54:19 *** aantonop has joined #bitcoin-core-dev
1712017-04-09T20:54:43 *** laurentmt has joined #bitcoin-core-dev
1722017-04-09T21:22:02 *** felco has joined #bitcoin-core-dev
1732017-04-09T22:01:09 *** marcoagner has joined #bitcoin-core-dev
1742017-04-09T22:02:09 <sipa> stevenroose: and comments are always welcome :)
1752017-04-09T22:26:05 *** aantonop has quit IRC
1762017-04-09T22:33:05 *** arowser has quit IRC
1772017-04-09T22:34:32 *** arowser has joined #bitcoin-core-dev
1782017-04-09T22:36:10 *** marcoagner has quit IRC
1792017-04-09T22:42:27 *** aantonop has joined #bitcoin-core-dev
1802017-04-09T22:46:56 *** netsin has joined #bitcoin-core-dev
1812017-04-09T22:48:34 *** netsin has quit IRC
1822017-04-09T23:09:40 *** bsm1175321 has joined #bitcoin-core-dev
1832017-04-09T23:09:51 *** jtimon has quit IRC
1842017-04-09T23:10:34 *** laurentmt has quit IRC
1852017-04-09T23:53:33 *** moctos has joined #bitcoin-core-dev
1862017-04-09T23:53:39 *** moctos_ has joined #bitcoin-core-dev
1872017-04-09T23:54:05 *** moctos has quit IRC
1882017-04-09T23:54:05 *** moctos_ has quit IRC
1892017-04-09T23:57:01 *** d9b4bef9 has quit IRC
1902017-04-09T23:58:08 *** d9b4bef9 has joined #bitcoin-core-dev