Backport missing PRs from Bitcoin 0.14 - Part 11 #1879

codablock · 2018-01-26T12:03:14Z

Continuation of #1770

This PR brings in most of the backports that were also merged into the Bitcoin 0.14 branch.

I decided to exclude bitcoin#10143 and bitcoin#10234 because these would require some nasty conflict resolution which will later (with 0.15) bring more merge conflicts.

c801c82 Move BIP70_MAX_PAYMENTREQUEST_SIZE to header (Pieter Wuille) 914fad1 Make KEY_SIZE a compile-time constant (Pieter Wuille)

3301587 boost: remove iostreams includes (Cory Fields)

ec1267f [wallet] Remove importmulti always-true check (Russell Yanofsky)

171fc91 Read/write mempool.dat as a binary. (Pavel Janík)

…open) 40f11f8 Fix for issue bitcoin#9775. Added check for open() returning a NULL pointer. (kirit93)

…lewallet mode 312c4f1 Fix segfault crash when shutdown the GUI in disablewallet mode (Jonas Schnelli)

…s are not successful e2e2f4c Return errors from importmulti if complete rescans are not successful (Russell Yanofsky)

…xoutproof() 9949ebf [Trivial] Remove incorrect help message from gettxoutproof() (John Newbery)

…ndow 874c736 Fix pruning test broken by 2 hour manual prune window (Russell Yanofsky)

fa4cd2e qa: Check return code when stopping nodes (MarcoFalke)

205830a build: add --enable-werror option (Cory Fields) b602fe0 build: warn about variable length arrays (Cory Fields)

… tests 75a1093 tests: Fix dangling pwalletMain pointer in wallet tests (Wladimir J. van der Laan) Tree-SHA512: 7fb6e8385fe7d542f9ecb113a08d675ca9e84907a1939b3a6ad41318fda55bc999b9bc8ffc3f56cd8610ca49d0db982d3c793df7bbcb7eb9638f4483030db6a8

d4ee7ba prevector: assert successful allocation (Cory Fields) c5f008a don't throw std::bad_alloc when out of memory. Instead, terminate immediately (Cory Fields) Tree-SHA512: 699ce8df5b1775a99c71d3cfc952b45da1c0091e1a4b6adfac52d5be6144c3d98f88ac3af90e5c73fff2f74666a499feb4a34434683ce5979814e869c0aeddc3

…n misconception f81f0d0 Update sendfrom RPC help to correct coin selection misconception (Russell Yanofsky)

fe71661 [doc] Update doc/bips.md for BIP90 implementation (Suhas Daftuar) Tree-SHA512: cfa72662e5e8ecedb7869b64c2064cc29d8736bcd9baacd59ea420dc2f93265c8d950e469863e77e8fa9348fbf1b39a1e2397253a2fd94b63adc909cc8c14873

83ac719 Change bitcoin address in RPC helpaddress to an invalid address, so people don't accidentally send coins there (like I did). (Marijn Stollenga) Tree-SHA512: ca1163466a149d567b97efbfcfa8fdfe2d474245b4dd5a1a92555b4e87f8e99df5fee4cd79ef1ce6a98db2337846af78f37c2e6b31d02008b11fa0e151ce6590

5c8fd50 Avoid VLA in hash.h (Pieter Wuille)

… keys 306bd72 Fix importmulti returning rescan errors for wrong keys (Russell Yanofsky) Tree-SHA512: ae9998236cbd3ff749d6b5c716bd76c9cec386b0708583e4912e4e05bf4584545258e1d0543aa5445024d2b5decf859a64f40c6503029773366a0f9a9ddf9b88

9e4d842 depends: make osx output deterministic (Cory Fields) Tree-SHA512: 997a671832a28e600bc3c9cd1340c6c1fa08f050d7cf676a5d51b38f08161ac418621520459a9a53dd87102c13fb69cb2e58ec3cc00f738e204eb787d6f0cad6

8c156b7 Bugfix: Only install manpages for built programs (Luke Dashjr) Tree-SHA512: f4c40910b257274a05d9c842b9adb5d9d01df7dd2d34a18bd2d98c78810dc3ed98ed12c08c55352e36a845f06ddb1efba9d4ecf5f1123d2c58406a5ecf589836

819b513 Add missing braces in semaphore posts in net (Matt Corallo) e007b24 Fix shutdown hang with >= 8 -addnodes set (Matt Corallo) Tree-SHA512: f2d7562bd5d333cd0e80562eb3b1fe329fc10ee713996e053d2ed669db6d9eb39550e0a6c6ab768cd070bfe92a5ea85e0f752470206706de196bd4a689b9bc07

05a9f22 Trivial: Fix typo in help getrawtransaction RPC (James Evans) Tree-SHA512: 22c68fb49771f96b94c482b28d7efc4d51737cbb973ed3954641f3ea1832c14e7b909030c132afebe17854da134f717acbf14ee26294c61a303bc33dc43aac4c

…ck before adding 4df76e2 Ensure an item exists on the rpcconsole stack before adding (Andrew Chow) Tree-SHA512: f3fd5e70da186949aff794f6e2ba122da2145331212dcc5e0595285bee9dc3aa6b400b15e8eeec4476099965b74f46c4ef80f8ed1e05d490580167b002b9a5e7

c624753 depends: fix zlib build on osx (Cory Fields) Tree-SHA512: fd9343edc24762fc4b7eb9798f0fb6f76d5f5aeef16a0bc3bab5400cb8f9b3ae8a2d34b480f03c853bb31ff4e39c267a2b81cd86df0532f11976072354aa9378

fa55853 rpc: Rename first named arg of createrawtransaction (MarcoFalke) Tree-SHA512: f2e07183f2503344e676e08fe0fd73e995d7c6fda3fc11c64116208dec8e445f0627583dfba85014129b6f2dc7e253b9d760e57e66811272db89e9ba25ce6dbc

19ecd1e Add tests for bitcoin-tx input checking (John Newbery) 21704f6 Check stderr when testing bitcoin-tx (John Newbery) eb66bf9 bitcoin-tx: Fix missing range check (Awemany) Tree-SHA512: 08c6153cf7dd5e0ecd23e24d81af4c0f17534d484179dd91dcd78d42df14c91284341d31cc695469a64c507bce72c34231748b7cabb7df8f1051d228fb0a62c5

… 32-bit w/ glibc 625488a util: Work around (virtual) memory exhaustion on 32-bit w/ glibc (Wladimir J. van der Laan) Tree-SHA512: 99b610a8cf9561998af90e16fc19320fddd30c987e8f33325d63df0f56d70235b94d9482e80f28154d4b33a3ecf4961686380c444ec18d1da5e8804a8b6f4de1

…ge mempools 011124a Update benchmarking with package statistics (Suhas Daftuar) 42cd8c8 Add benchmarking for CreateNewBlock (Suhas Daftuar) eed816a Mining: return early when block is almost full (Suhas Daftuar) Tree-SHA512: c0d8f71e4e0441acf3f4ca12f8705e413b59b323659346a447145653def71710537fb4c6d80cad8e36d68b0aabf19c92e9eab7135a8897b053ed58720856cdda

4b87973 [rpc] Remove auth cookie on shutdown (practicalswift) Tree-SHA512: 5d7d04413d3eb6a8d167443a3fdfe4c289bdc5cbc4c9b257a557c2a7fab5d4b32a81a1357e27b77ce2623e76e495b2e8b21eab0ab032bb261fdcdf171bac1dac

30f30c0 Add braces to submitblock per current style. (Gregory Maxwell) 4f15ea1 Check transaction count early in submitblock. (Gregory Maxwell) ada0caa Make GetWitnessCommitmentIndex callable on blocks without a coinbase txn. (Gregory Maxwell) Tree-SHA512: 02dcd337ad9cdd8e4fa6a42c009d016026d1229c193676ed6fcc9ce55e924fedec57f516ac1e95c3db0985243ba908307338ce783a70416cb292bed881002bfc

… potentially referencing c36ea69 [wallet] Make sure pindex is non-null before possibly referencing in LogPrintf call. (Karl-Johan Alm) Tree-SHA512: a14c9f3e1228bca91977bea821c56a377d80889b41d250050c9be67aa93e460319a7cf1d4b63ee40f23b5a34159590f0e3fe15dc88698dc694c0c8098bd2db4d

…ys/values 6c914ac [wallet] Securely erase potentially sensitive keys/values (Thomas Snider) Tree-SHA512: 071d88c4093108d4e4eced35a6ffcebe3f499798194f5b1be661ffa5b78b5f55311667f6d2a72758d85290f61f958381ee95d380b9045ca18e9e1875f0e686c8

661caf8 [doc] Minor corrections to osx dependencies (fanquake) Tree-SHA512: 61451807d6208002550d2e37d26094a5515b11daacab39eb508b6168eebcea86f93dcc7f853aab490baec3727740c90d404c481e11b3d03dcea9f76f8fc7be12

…wallet by default afc693d contrib/init/bitcoind.openrcconf: Don't disable wallet by default (Luke Dashjr) Tree-SHA512: 1763a9e91788485b079b96239cf09f1557b338e5045aa4ffbad3908f88c4e362b9b5d86a8a0f33734899de244e76e7ced02a6be8e52b3fb69258a5101d6445ef

dd07068 Fixed typo in documentation for merkleblock.h (Mikerah) Tree-SHA512: 92655c8022eb33852c116da517b12bd17b3b668713cd85fe185a15245ea9810605626b1d0955fa117f7b56863e0a52b3a86dab42409332452b5eb72a7d34f30d

3503716 Trivial: remove extra character from comment (CryptAxe) Tree-SHA512: 66ee474945b4f9fd098cee0d031c1860c0777d4b05c609c0eaba32234209356b630766de30235b49d22cc5ea7cf2bb93410dea4466396369a0cdc0b631512a33

ac9cd95 contrib: Update location of seeds.txt (Wladimir J. van der Laan) Tree-SHA512: c12a75787ba87f03707c21731da083b466762a7e0af9ca501107695ea1074025907cc24805c7c87f4a66daa7f4f13e574da16be1681e61deaf1acbd72176b3ff

…mpool help. 3a0a5bc [doc] Add hint about getmempoolentry to getrawmempool help. (Karl-Johan Alm) Tree-SHA512: 8327d7d7ad93296525fbf95b7a824e3525bde84653999f125afd845823eb39e3a03cd39725962ed949aa2b9ad207ecad6d287294fa321ff1a4d7fbd5a4b8560b

Until Bitcoin bitcoin#9908 gets backported

UdjinM6

👍, will test

UdjinM6 · 2018-01-26T12:16:53Z

src/rpc/misc.cpp

            + HelpExampleCli("walletpassphrase", "\"mypassphrase\" 30") +
            "\nCreate the signature\n"
-            + HelpExampleCli("signmessage", "\"XwnLY9Tf7Zsef8gMGL2fhWA9ZmMjt4KPwg\" \"my message\"") +
+            + HelpExampleCli("signmessage", "\"XwnLY9Tf7Zsef8gMGL2fhWA9ZmMjt4KPwG\" \"my message\"") +


there are more examples that should be fixed in a similar way

How did the invalid one make it into the comment anyway?

The invalid ones are intentional so that people don't send Dash to it by accident. This is part of bitcoin#9865 (but with Dash addresses)

UdjinM6

tests fail, see below

UdjinM6 · 2018-01-26T13:28:48Z

src/test/data/bitcoin-util-test.json

+    "error_txt": "error: TX output missing or too many separators",
+    "description": "Malformed outaddr argument (no address specified). Expected to fail."
+  },
+  { "exec": "./bitcoin-tx",


s/bitcoin/dash/g

UdjinM6 · 2018-01-26T13:30:22Z

src/test/data/bitcoin-util-test.json

+  { "exec": "./bitcoin-tx",
+    "args":
+    ["-create",
+     "outaddr=1:13tuJJDR2RgArmgfv6JScSdreahzgc4T6o:garbage"],


address should be dashified

UdjinM6 · 2018-01-26T13:42:25Z

src/miner.cpp


-            if (nConsecutiveFailed > MAX_CONSECUTIVE_FAILURES && nBlockWeight >
-                    nBlockMaxWeight - 4000) {
+            if (nConsecutiveFailed > MAX_CONSECUTIVE_FAILURES && nBlockSize > nBlockMaxSize - 4000) {


should probably also divide by 4 when converting from weight to block size i.e. s/4000/1000/

Also use S flag instead of W flag for the outpubkey test. Not really needed to fix the test, but we don't have the W flag in Dash so lets not confuse people.

codablock · 2018-01-29T10:27:52Z

I replaced all the example Dash addresses I could find with invalid ones, except the ones about txindex and related.

UdjinM6 · 2018-01-29T19:24:43Z

Not sure what you mean by

except the ones about txindex and related

I see this:

> grep XwnLY9Tf7Zsef8gMGL2fhWA9ZmMjt4KPwg src/*/*.cpp
src/rpc/misc.cpp:            + HelpExampleCli("validateaddress", "\"XwnLY9Tf7Zsef8gMGL2fhWA9ZmMjt4KPwg\"")
src/rpc/misc.cpp:            + HelpExampleRpc("validateaddress", "\"XwnLY9Tf7Zsef8gMGL2fhWA9ZmMjt4KPwg\"")
src/rpc/misc.cpp:            + HelpExampleCli("getaddressmempool", "'{\"addresses\": [\"XwnLY9Tf7Zsef8gMGL2fhWA9ZmMjt4KPwg\"]}'")
src/rpc/misc.cpp:            + HelpExampleRpc("getaddressmempool", "{\"addresses\": [\"XwnLY9Tf7Zsef8gMGL2fhWA9ZmMjt4KPwg\"]}")
src/rpc/misc.cpp:            + HelpExampleCli("getaddressutxos", "'{\"addresses\": [\"XwnLY9Tf7Zsef8gMGL2fhWA9ZmMjt4KPwg\"]}'")
src/rpc/misc.cpp:            + HelpExampleRpc("getaddressutxos", "{\"addresses\": [\"XwnLY9Tf7Zsef8gMGL2fhWA9ZmMjt4KPwg\"]}")
src/rpc/misc.cpp:            + HelpExampleCli("getaddressdeltas", "'{\"addresses\": [\"XwnLY9Tf7Zsef8gMGL2fhWA9ZmMjt4KPwg\"]}'")
src/rpc/misc.cpp:            + HelpExampleRpc("getaddressdeltas", "{\"addresses\": [\"XwnLY9Tf7Zsef8gMGL2fhWA9ZmMjt4KPwg\"]}")
src/rpc/misc.cpp:            + HelpExampleCli("getaddressbalance", "'{\"addresses\": [\"XwnLY9Tf7Zsef8gMGL2fhWA9ZmMjt4KPwg\"]}'")
src/rpc/misc.cpp:            + HelpExampleRpc("getaddressbalance", "{\"addresses\": [\"XwnLY9Tf7Zsef8gMGL2fhWA9ZmMjt4KPwg\"]}")
src/rpc/misc.cpp:            + HelpExampleCli("getaddresstxids", "'{\"addresses\": [\"XwnLY9Tf7Zsef8gMGL2fhWA9ZmMjt4KPwg\"]}'")
src/rpc/misc.cpp:            + HelpExampleRpc("getaddresstxids", "{\"addresses\": [\"XwnLY9Tf7Zsef8gMGL2fhWA9ZmMjt4KPwg\"]}")
src/wallet/rpcwallet.cpp:            + HelpExampleCli("listunspent", "6 9999999 \"[\\\"XwnLY9Tf7Zsef8gMGL2fhWA9ZmMjt4KPwg\\\",\\\"XuQQkwA4FYkq2XERzMY2CiAZhJTEDAbtcg\\\"]\"")
src/wallet/rpcwallet.cpp:            + HelpExampleRpc("listunspent", "6, 9999999 \"[\\\"XwnLY9Tf7Zsef8gMGL2fhWA9ZmMjt4KPwg\\\",\\\"XuQQkwA4FYkq2XERzMY2CiAZhJTEDAbtcg\\\"]\"")

None of these sends funds or uses privkey in any way though, so it's safe to use valid addresses there I think.

UdjinM6

slightly tested ACK

Backported 10228 twice - in dashpay#1879 and dashpay#2911

Backported 10228 twice - in #1879 and #2911

Backported 10228 twice - in dashpay#1879 and dashpay#2911

revert #3036, #2911, #1879, #1432: revert dashification of filenames

laanwj and others added 30 commits January 26, 2018 12:59

Merge bitcoin#9785: Avoid variable length arrays

5e59642

c801c82 Move BIP70_MAX_PAYMENTREQUEST_SIZE to header (Pieter Wuille) 914fad1 Make KEY_SIZE a compile-time constant (Pieter Wuille)

Merge bitcoin#9786: boost: remove iostreams includes

3ab5fab

3301587 boost: remove iostreams includes (Cory Fields)

Merge bitcoin#9760: [wallet] Remove importmulti always-true check

199b07a

ec1267f [wallet] Remove importmulti always-true check (Russell Yanofsky)

Merge bitcoin#9813: Read/write mempool.dat as a binary.

8d2edc4

171fc91 Read/write mempool.dat as a binary. (Pavel Janík)

Merge bitcoin#9798: Fix Issue bitcoin#9775 (Check returned value of f…

15a035b

…open) 40f11f8 Fix for issue bitcoin#9775. Added check for open() returning a NULL pointer. (kirit93)

Merge bitcoin#9817: Fix segfault crash when shutdown the GUI in disab…

0a6f63b

…lewallet mode 312c4f1 Fix segfault crash when shutdown the GUI in disablewallet mode (Jonas Schnelli)

Merge bitcoin#9773: Return errors from importmulti if complete rescan…

ac981ec

…s are not successful e2e2f4c Return errors from importmulti if complete rescans are not successful (Russell Yanofsky)

Merge bitcoin#9711: [Trivial] Remove incorrect help message from gett…

0e8be43

…xoutproof() 9949ebf [Trivial] Remove incorrect help message from gettxoutproof() (John Newbery)

Merge bitcoin#9820: Fix pruning test broken by 2 hour manual prune wi…

f09ae21

…ndow 874c736 Fix pruning test broken by 2 hour manual prune window (Russell Yanofsky)

Merge bitcoin#9824: qa: Check return code when stopping nodes

d880308

fa4cd2e qa: Check return code when stopping nodes (MarcoFalke)

Merge bitcoin#9789: build: add --enable-werror and warn on vla's

af006a3

205830a build: add --enable-werror option (Cory Fields) b602fe0 build: warn about variable length arrays (Cory Fields)

Merge bitcoin#9840: Update sendfrom RPC help to correct coin selectio…

7c3b8ec

…n misconception f81f0d0 Update sendfrom RPC help to correct coin selection misconception (Russell Yanofsky)

Merge bitcoin#9791: Avoid VLA in hash.h

16b2a1d

5c8fd50 Avoid VLA in hash.h (Pieter Wuille)

Merge bitcoin#9891: depends: make osx output deterministic

818fc1e

9e4d842 depends: make osx output deterministic (Cory Fields) Tree-SHA512: 997a671832a28e600bc3c9cd1340c6c1fa08f050d7cf676a5d51b38f08161ac418621520459a9a53dd87102c13fb69cb2e58ec3cc00f738e204eb787d6f0cad6

Merge bitcoin#9892: Bugfix: Only install manpages for built programs

bd0b0da

8c156b7 Bugfix: Only install manpages for built programs (Luke Dashjr) Tree-SHA512: f4c40910b257274a05d9c842b9adb5d9d01df7dd2d34a18bd2d98c78810dc3ed98ed12c08c55352e36a845f06ddb1efba9d4ecf5f1123d2c58406a5ecf589836

Merge bitcoin#10037: Trivial: Fix typo in help getrawtransaction RPC

edd514c

05a9f22 Trivial: Fix typo in help getrawtransaction RPC (James Evans) Tree-SHA512: 22c68fb49771f96b94c482b28d7efc4d51737cbb973ed3954641f3ea1832c14e7b909030c132afebe17854da134f717acbf14ee26294c61a303bc33dc43aac4c

Merge bitcoin#9973: depends: fix zlib build on osx

a81f445

c624753 depends: fix zlib build on osx (Cory Fields) Tree-SHA512: fd9343edc24762fc4b7eb9798f0fb6f76d5f5aeef16a0bc3bab5400cb8f9b3ae8a2d34b480f03c853bb31ff4e39c267a2b81cd86df0532f11976072354aa9378

Merge bitcoin#10139: [rpc] Remove auth cookie on shutdown

0fc9b2b

4b87973 [rpc] Remove auth cookie on shutdown (practicalswift) Tree-SHA512: 5d7d04413d3eb6a8d167443a3fdfe4c289bdc5cbc4c9b257a557c2a7fab5d4b32a81a1357e27b77ce2623e76e495b2e8b21eab0ab032bb261fdcdf171bac1dac

laanwj and others added 13 commits January 26, 2018 12:59

Merge bitcoin#10260: [doc] Minor corrections to osx dependencies

b0a8172

661caf8 [doc] Minor corrections to osx dependencies (fanquake) Tree-SHA512: 61451807d6208002550d2e37d26094a5515b11daacab39eb508b6168eebcea86f93dcc7f853aab490baec3727740c90d404c481e11b3d03dcea9f76f8fc7be12

Merge bitcoin#10258: Fixed typo in documentation for merkleblock.h

ed898f1

dd07068 Fixed typo in documentation for merkleblock.h (Mikerah) Tree-SHA512: 92655c8022eb33852c116da517b12bd17b3b668713cd85fe185a15245ea9810605626b1d0955fa117f7b56863e0a52b3a86dab42409332452b5eb72a7d34f30d

Merge bitcoin#10309: Trivial: remove extra character from comment

631d9f9

3503716 Trivial: remove extra character from comment (CryptAxe) Tree-SHA512: 66ee474945b4f9fd098cee0d031c1860c0777d4b05c609c0eaba32234209356b630766de30235b49d22cc5ea7cf2bb93410dea4466396369a0cdc0b631512a33

Merge bitcoin#10495: contrib: Update location of seeds.txt

e9976d2

ac9cd95 contrib: Update location of seeds.txt (Wladimir J. van der Laan) Tree-SHA512: c12a75787ba87f03707c21731da083b466762a7e0af9ca501107695ea1074025907cc24805c7c87f4a66daa7f4f13e574da16be1681e61deaf1acbd72176b3ff

Use nBlockSize/nBlockMaxSize instead of nBlockWeight/nBlockMaxWeight

d500d77

Include test_dash.h instead of test_bitcoin.h

4d3856b

Use constant 7200 instead of TIMESTAMP_WINDOW for now

5901531

Until Bitcoin bitcoin#9908 gets backported

Fix rpc tests

a4430b6

Fix formatting of help in sendfrom

4959ff2

UdjinM6 requested changes Jan 26, 2018

View reviewed changes

UdjinM6 added this to the 12.3 milestone Jan 26, 2018

UdjinM6 requested changes Jan 26, 2018

View reviewed changes

UdjinM6 reviewed Jan 26, 2018

View reviewed changes

codablock added 3 commits January 26, 2018 17:29

Fix new dash-tx tests

873bd6c

Also use S flag instead of W flag for the outpubkey test. Not really needed to fix the test, but we don't have the W flag in Dash so lets not confuse people.

Give 1000 bytes instead of 4000 bytes before the block is full

7e4e4ea

Replace more example Dash addresses with invalid addresses

c8e5c40

UdjinM6 approved these changes Jan 29, 2018

View reviewed changes

UdjinM6 merged commit 0e6672b into dashpay:develop Jan 29, 2018

codablock deleted the pr_backport_bitcoin_0.14-11 branch January 29, 2018 21:15

UdjinM6 added a commit to UdjinM6/dash that referenced this pull request May 28, 2019

Remove duplicate code from src/Makefile.am

d9141f0

Backported 10228 twice - in dashpay#1879 and dashpay#2911

UdjinM6 mentioned this pull request May 28, 2019

Remove duplicate code from src/Makefile.am #2944

Merged

UdjinM6 added a commit that referenced this pull request May 28, 2019

Remove duplicate code from src/Makefile.am (#2944)

5780fa6

Backported 10228 twice - in #1879 and #2911

barrystyle pushed a commit to PACGlobalOfficial/PAC that referenced this pull request Jan 22, 2020

Remove duplicate code from src/Makefile.am (dashpay#2944)

0c6a66d

Backported 10228 twice - in dashpay#1879 and dashpay#2911

PastaPastaPasta added a commit that referenced this pull request Aug 11, 2022

Merge pull request #4478 from kittywhiskers/undashify

f83d857

revert #3036, #2911, #1879, #1432: revert dashification of filenames

Backport missing PRs from Bitcoin 0.14 - Part 11 #1879

Backport missing PRs from Bitcoin 0.14 - Part 11 #1879

Uh oh!

Conversation

codablock commented Jan 26, 2018

Uh oh!

UdjinM6 left a comment

Choose a reason for hiding this comment

Uh oh!

UdjinM6 Jan 26, 2018

Choose a reason for hiding this comment

Uh oh!

schinzelh Jan 26, 2018

Choose a reason for hiding this comment

Uh oh!

codablock Jan 26, 2018

Choose a reason for hiding this comment

Uh oh!

schinzelh Jan 26, 2018

Choose a reason for hiding this comment

Uh oh!

UdjinM6 left a comment

Choose a reason for hiding this comment

Uh oh!

UdjinM6 Jan 26, 2018

Choose a reason for hiding this comment

Uh oh!

UdjinM6 Jan 26, 2018

Choose a reason for hiding this comment

Uh oh!

UdjinM6 Jan 26, 2018

Choose a reason for hiding this comment

Uh oh!

codablock commented Jan 29, 2018

Uh oh!

UdjinM6 commented Jan 29, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

UdjinM6 left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

UdjinM6 commented Jan 29, 2018 •

edited

Loading