< MarcoFalke>
Does the modal overlay work with -reindex?
< wumpus>
is libconsensus work still underway in an active enough fashion to warrant a project in https://github.com/bitcoin/bitcoin/projects? I haven't heard of it in quite a while. It may have been because of sidetracking due to segwit ofcourse.
< GitHub25>
bitcoin/master faa91bc MarcoFalke: CONTRIBUTING: Mention not to open several pulls
< GitHub25>
bitcoin/master bc9e3ab Wladimir J. van der Laan: Merge #8771: CONTRIBUTING: Mention not to open several pulls...
< GitHub70>
[bitcoin] laanwj closed pull request #8771: CONTRIBUTING: Mention not to open several pulls (master...Mf1609-ContributeDoc) https://github.com/bitcoin/bitcoin/pull/8771
< GitHub152>
bitcoin/master b194872 fanquake: Remove old manpages from contrib/debian
< GitHub152>
bitcoin/master bae178f Wladimir J. van der Laan: Merge #8743: Remove old manpages from contrib/debian in favour of doc/man...
< GitHub12>
[bitcoin] laanwj closed pull request #8743: Remove old manpages from contrib/debian in favour of doc/man (master...remove-old-manpages) https://github.com/bitcoin/bitcoin/pull/8743
< btcdrak>
wumpus: a project would help significantly.
< wumpus>
any idea which issues and pulls to add to it?
< jonasschnelli>
Yes. Was also just stumpled over that page.
< jonasschnelli>
So,.. false alarm. :)
< jonasschnelli>
But I was suppriesed that you can upload a zip file to github containing a +x file
< wumpus>
yep, but better be safe than sorry, it would not have been the first time someone tried to upload a questionable file to the repository with an issue
< morcos>
cfields_: jeremyrubin: Is there room for some speedup by using move semantics on CCoinsCacheEntry inside of CCoinsViewCache:BatchWrite? We're always deleting the cache version anyway.
< sipa>
morcos: i've been thinking for a while about another coinsclviewcache like structure, that contains updates which are likely written... which writes the changes directly to the parent, and the old value in the subcache, only to be used whenreverting
< morcos>
sipa: hmm.. interesting. is this in lieu of a potential switch to a non-txid based structure, or just an easier change for now
< morcos>
sipa: i think we have a slew of performance improvements which aren't too far off from PR'ing.. one is based on keeping a consistent hot cache on top of pcoinstip... i think if the changes to that are just moves to pcoinstip, it should be fast, b/c the time to populate it is amortized. the trick is not erasing all the rest of it
< sipa>
morcos: yeah, i don't have time to work on any of this, so don't let it interfere with ongoing work
< sipa>
just throwing it out there as an idea
< morcos>
sipa: do you think the move idea has any merit? seems like it might not be a large change, just c++11 i'm not very familiar with
< sipa>
morcos: no idea what it refers to, i haven't been following the past days (busy with moving)
< sipa>
but if it's simple and it helps, i'm all for it
< morcos>
sipa: i was just suggesting above, that if we make a way to std:move CCoinsCacheEntry's we can do that inside of BatchWrite instead of swapping the the underlying coins
< sipa>
morcos: that simplifies the code, but it doesn't have much of a performance impact
< sipa>
swap and move assign are often the same thing
< morcos>
sipa: ok, that's what i was wondering.
< morcos>
the erasing of the old cache is just so painful (performance wise)..
< Chris_Stewart_5>
Is there a better way to test serialization symmetry for something that implementions SerializationOp? This seems like a hacky way to do it
< MarcoFalke>
Could be just travis having messed up the osx environment. (Seems to happen regularly)
< luke-jr>
MarcoFalke: the BIP process does not actually allow for amending BIP 1
< wumpus>
I think it does, BIP1 doesn't have final status
< MarcoFalke>
In the text of BIP1 it says it is never meant to be completed, so you can always change it
< wumpus>
yes
< MarcoFalke>
If it says somewhere in BIP1 that it is not allowed to amend, just remove that line :P. Chicken-egg problem solved
< wumpus>
I'm quite sure it doesn't say that
< GitHub70>
[bitcoin] laanwj opened pull request #8811: rpc: Add support for JSON-RPC named arguments (master...2016_09_rpc_named_arguments) https://github.com/bitcoin/bitcoin/pull/8811