💬 Sjors commented on pull request "qt: drop unused watch-only functionality":
(https://github.com/bitcoin/bitcoin/pull/32459#issuecomment-2871909501)
Somehow the Date column disappeared.

✅ hebasto closed a pull request: "Add used balance to overview page"
(https://github.com/bitcoin-core/gui/pull/775)

💬 hebasto commented on pull request "Add used balance to overview page":
(https://github.com/bitcoin-core/gui/pull/775#issuecomment-2871965508)
Closing due to inactivity. Feel free to re-open.

💬 hebasto commented on issue "`used` balance should be shown on overview page":
(https://github.com/bitcoin-core/gui/issues/769#issuecomment-2871972651)
Cross-posting @luke-jr's [comment](https://github.com/bitcoin-core/gui/pull/775#issuecomment-2073729945) from https://github.com/bitcoin-core/gui/pull/775:

> What is the use case here? (And the avoid_reuse flag in general...)

💬 laanwj commented on pull request "cmake: Allow `WITH_DBUS` on all Unix-like systems":
(https://github.com/bitcoin/bitcoin/pull/32469#issuecomment-2871975207)
ACK 5b7ed460c7c181f1fd34a927a09aed36284083cb

This is conceptually correct because freedesktop DBUS is a standard for UNIX-like systems, not specifically Linux.

👍 TheCharlatan approved a pull request: "threading: drop CSemaphore in favor of c++20 std::counting_semaphore"
(https://github.com/bitcoin/bitcoin/pull/32466#pullrequestreview-2832753782)
ACK 6f7052a7b96f058568af9aed2f014ae7a25e0f68

👍 TheCharlatan approved a pull request: "guix: move `*-check.py` scripts under contrib/guix/"
(https://github.com/bitcoin/bitcoin/pull/32458#pullrequestreview-2832760013)
ACK 415650cea94f5050d7c368fdf9fd9878809957e1

Guix build matches.

🤔 hebasto reviewed a pull request: "qt: drop unused watch-only functionality"
(https://github.com/bitcoin/bitcoin/pull/32459#pullrequestreview-2832784295)
Are these lines still necessary:https://github.com/bitcoin/bitcoin/blob/59e09e0fb7b4cf8f8db97c2f81a8f6f69fe6cacf/src/interfaces/wallet.h#L374-L377 ?

💬 donaldevinev1 commented on pull request "Remove arbitrary limits on OP_Return (datacarrier) outputs":
(https://github.com/bitcoin/bitcoin/pull/32359#issuecomment-2872047967)
Concept NACK - removing these limits is like saying 'because some cars are speeding we should remove the speed limit on all the roads'.

💬 hebasto commented on pull request "qt: drop unused watch-only functionality":
(https://github.com/bitcoin/bitcoin/pull/32459#discussion_r2084441564)
Where is this value used?

💬 Sjors commented on pull request "qt: drop unused watch-only functionality":
(https://github.com/bitcoin/bitcoin/pull/32459#discussion_r2084444051)
It's not, dropping it again.

💬 pinheadmz commented on pull request "Remove arbitrary limits on OP_Return (datacarrier) outputs":
(https://github.com/bitcoin/bitcoin/pull/32359#issuecomment-2872113690)
> Concept NACK - removing these limits is like saying 'because some cars are speeding we should remove the speed limit on all the roads'.

It's more like "some cars are driving idiotically slow so we'll allow them to drive on the shoulder so they bother less people"

💬 Sjors commented on pull request "qt: drop unused watch-only functionality":
(https://github.com/bitcoin/bitcoin/pull/32459#issuecomment-2872115535)
> Are these lines still necessary:

I got rid of them.

For some reason QT is hiding column 1, which happens to be the date, but it also happens if I e.g. swap the date and type column.

💬 hebasto commented on pull request "qt: drop unused watch-only functionality":
(https://github.com/bitcoin/bitcoin/pull/32459#issuecomment-2872139997)
> > Are these lines still necessary:
>
> I got rid of them.
>
> For some reason QT is hiding column 1, which happens to be the date, but it also happens if I e.g. swap the date and type column.
>
> Aha, QT caches column layout between sessions, so it remembers how this column behaved on an earlier run.

^ Indeed.

💬 Sjors commented on pull request "qt: drop unused watch-only functionality":
(https://github.com/bitcoin/bitcoin/pull/32459#issuecomment-2872192620)
I renamed `TransactionViewHeaderState` so that it's reset. This prevents hiding the date column.

💬 hebasto commented on pull request "qt: drop unused watch-only functionality":
(https://github.com/bitcoin/bitcoin/pull/32459#issuecomment-2872198375)
> I renamed `TransactionViewHeaderState` so that it's reset. This prevents hiding the date column.

A more general approach would involve versioning, though.

👋 Sjors's pull request is ready for review: "qt: drop unused watch-only functionality"
(https://github.com/bitcoin/bitcoin/pull/32459)

💬 Sjors commented on pull request "qt: drop unused watch-only functionality":
(https://github.com/bitcoin/bitcoin/pull/32459#issuecomment-2872212304)
Added a release note.

> A more general approach would involve versioning

I'm open to suggestions. Though this screen is unlikely to change again between now and when we switch to QML.

💬 Zakonduasabbat commented on issue "Bitcoin Logo/Icon Issues":
(https://github.com/bitcoin-core/gui/issues/200#issuecomment-2872268536)
Wow

💬 hebasto commented on pull request "wallet: init, don't error out when loading legacy wallets":
(https://github.com/bitcoin/bitcoin/pull/32449#issuecomment-2872272437)
Concept ACK.

💬 l0rinc commented on pull request "validation: periodically flush dbcache during reindex-chainstate":
(https://github.com/bitcoin/bitcoin/pull/32414#issuecomment-2872303093)
> I'll retry it with 100 minutes to see if it would fix the situation.

With 100 mins it's **7%** slower instead of **11%** - still quite bad, but we shouldn't have to choose between performance and persistence, so let's find ways to reduce that. And IBD is the usecase that matters most anyway.

<details>
<summary>Details</summary>

```bash
COMMITS="59d3e4ed34eb55cb40928d524cb0bd5e183ed85a cf75a7b599fe459935b7aa4776e91526fcfb0fcc"; \
STOP_HEIGHT=888888; DBCACHE=45000; \
CC=gcc; CXX=g+
...