₿ Bitcoin Core PR Review Club en Español

Pregunta 8

wallet: Add importdescriptors interface by polespinasa · Pull Requ...

Pol Espinasa 31/03/2026 11:15
Pregunta 8 -> Hay algunos comentarios que sugieren mover ImportDescriptors() a fuera de CWallet y crear un nuevo fichero. ¿Qué hace CWallet? ¿Por qué la función no necesita estar dentro de CWallet realmente?
08/04/2026 17:00
🤓
> 08/04/2026 17:01

CWallet es la clase principal para Wallets en Bitcoin Core. Dentro de sus responsabilidades están: Trackear balances, firmar transacciones, manejar descriptors y keys, etc.

Según achow101 (https://github.com/bitcoin/bitcoin/pull/34861#discussion_r2998212485) la idea es no extender la clase (o tal vez los ficheros wallet.cpp o wallet.h) más de lo que está.

Pol Espinasa 08/04/2026 17:01
Correcto, un argumento es que el fichero ya es muy largo y añadir mas complegidad hace que sea mas dificil de leer
👍🏻 2
08/04/2026 17:02
El otro motivo es que dicha funcion no usa ningun metodo privado de CWallet por lo que no es necesario que este dentro de la clase
👍🏻 1
08/04/2026 17:02
La funcion puede hacer todo lo que quiere hacer con metodos publicos
08/04/2026 17:03
pasar un objeto CWallet por referencia a la funcion es mas que suficiente para importar un descriptor