Pregunta 7

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

Pol Espinasa 31/03/2026 11:15
Pregunta 7 -> En la classe WalletImpl se añade un método privado ProcessDescriptortImport() que solo devuelve la respuesta de otra función. ¿Por qué crees que se ha hecho esto y no se llama la función ImportDescriptor() directamente desde importDescriptors()? Lo habrías hecho igual?
08/04/2026 16:50
👀
arejula27 08/04/2026 16:52
entiendo que es un wrapper que nos permte coger el mutex
08/04/2026 16:52
de tal forma que ImportDescriptor no se procupa por la concurrecnia solo funcionalidad y asi es mas reultilizable y facil de testear
08/04/2026 16:53
ademas que asi puede consumir ImportDescriptorRequest
> 08/04/2026 16:54
No lo tengo 100% claro pero sospecho que es para aplicar EXCLUSIVE_LOCKS_REQUIRED, yo no agregaría un campo privado, por consistencia lo saco de la clase y lo convierto en una función libre como MakeWalletTxOut or MakeWalletTx.
08/04/2026 16:55
el mutex hay otras formas de gestionarlo, la respuesta era un poco mas sencilla
08/04/2026 16:55
era para mantener el codigo como hace RPC 🙂
08/04/2026 16:55
misma estructura asi sea facil de revisar al comparar
👍 1
08/04/2026 16:56
a parte por un tema de estilo es mas facil de leer ya que puedes usar request sin tener que ir atributo por atributo
arejula27 08/04/2026 16:56
si, eso he pensado al final
Pol Espinasa 08/04/2026 16:56
pero es probable que eso cambie por lo que vamos a ver en la siguiente pregunta
👍🏻 1
08/04/2026 16:57
Ese comment no lo entiendo mucho… no deberían mantenerse absolutamente todos los metodos como publicos?
Pol Espinasa 08/04/2026 16:58
processDescriptorImport solo lo deberia poder usar WalletImpl en esta version del PR
08/04/2026 16:58
por lo que el metodo se hace privado
> 08/04/2026 16:59
CWallet no?
08/04/2026 16:59
si quieres avanzamos igual
Pol Espinasa 08/04/2026 16:59
processDescriptorImport esta en WalletImpl
08/04/2026 17:00
si avanzamos a la 8 y si tienes mas dudas dispara por aqui y te respondo mas tarde cuando vuelva