Pregunta 5

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

Pol Espinasa 31/03/2026 11:14
Pregunta 5 -> El PR introduce una nueva “struct” llamada ImportDescriptorResult, posteriormente en la función ProcessDescriptortImport() se usa una función “switch” para traducir los códigos de error a RPC. ¿Por qué ImportDescriptorResult no devuelve directamente los códigos de error correctos?
08/04/2026 16:34
Vamos ya con preguntas mas interesantes relacionadas con el Codigo
> 08/04/2026 16:35
Porque es responsabilidad del consumidor, en este caso la capa RPC.
08/04/2026 16:35
Precisamente lo que el PR busca resolver es esa mezcla de responsabilidades
Pol Espinasa 08/04/2026 16:36
Correcto de ahi la extracción de todo ese codigo fuera del codigo RPC
👍 2
08/04/2026 16:36
No queremos que las interfaces tengan que conocer RPC ni que RPC tenga que conocer las interfaces. De la misma forma el codigo que se ha añadido dentro de CWallet no tiene porque saber sobre codigos de error de RPC.
08/04/2026 16:37
Lo que hace la propuesta pues es crear unos codigos de error genéricos y ya luego cada interfaz se encarga de traducirlas si es necesario
08/04/2026 16:37
Algo que añadir aqui? Saltamos a la pregunta 6?
arejula27 08/04/2026 16:38
una duda
Pol Espinasa 08/04/2026 16:38
dispara
arejula27 08/04/2026 16:39
hace tiempo se mergeo el tipo expect para gestion de errores y se esta discutiendo el result para errores mas “user friendly”
08/04/2026 16:39
entiendo que en esta PR no es necesario ninguno no?
08/04/2026 16:39
(aunque no se haya mergado para la discusion imaginemos que si)
Pol Espinasa 08/04/2026 16:40
desconozco esto sobre los errores pero hay un comentario de Achow que menciona result.Error()
08/04/2026 16:40
te refieres a esto?
arejula27 08/04/2026 16:40
si
08/04/2026 16:41
Tengo que mirar aun que es eso, pero seguramente tomare esa sugerencia ya que quedara mas limpio y corto.
🔥 1
arejula27 08/04/2026 16:41
lo digo por que como ya has creado un tipo result, igual se solapan responsabilidades
Pol Espinasa 08/04/2026 16:41
tengo que mirar en detalle como funciona tbh
08/04/2026 16:41
lo desconozco
arejula27 08/04/2026 16:42
vale sin problema
08/04/2026 16:42
como hint, es el mimso comportamineto que expect en c++ std 23, para cuando lo mires
Pol Espinasa 08/04/2026 16:42
si se solapa seguramente usare lo que ya esta implementado. Si no se solapa y falta algo; seguramente ImportDescriptorResult tendra un atributo de ese tipo para la gestion de errores
👍 2
08/04/2026 16:43
Saltamos a la 6 que se nos come el tiempo!
arejula27 08/04/2026 16:43
sii
> 08/04/2026 16:44
Buena pregunta @arejula27 no sabía acerca ese dato!
❤️ 1