Pregunta 10

Si:

1. In 1571cdb00517a62769f635e077581c2095289646 “refactor: Add ImportDescriptor to CWallet”:

La variable is_ranged lo convertiría a un bool puro, no le veo sentido mantenerlo como optional ya que nunca pasa a ser false, sus 2 valores posibles son nullopt y true.

2. In 1571cdb00517a62769f635e077581c2095289646 “refactor: Add ImportDescriptor to CWallet”:

El hecho que next_idx y next_index existan bajo el mismo scope me confunde un poco, es un nit pero renombraría next_idx a requested_next_index para mas claridad.

3. In b88462d758e2d24f278607dc5fbd9dcee6c1a076 “wallet: Add importdescriptors interface”:

Si importDescriptors se mantiene dentro de WalletImpl yo sacaría el metodo privado processDescriptorImport y lo mantendría como una función libre, siguiendo el mismo patrón que MakeWalletTx o MakeWalletTxOut

por que:

    std::optional<bool> active;
    std::optional<bool> internal;

son opcionales y no bool con un valor por defecto? no se esta creando logica innecsaria? (caso true, caso false, caso empty)