Pregunta 10

Pregunta 10 -> En CWalletTx::Update, ¿por qué RecomputeCanonical() solo se llama cuando !isConfirmed()? ¿Qué saldría mal si se llamara incondicionalmente?

Una vez está confirmada una siempre es la canonica

RecomputeCanonical puede eliger la mejor por peso sin tener en cuenta si alguna variante está confirmada

correcto y eso seria un problema

¿no tendría sentido poner el if (!confirmed) dentro de la función?

pero update se llama dentro de esa call?

Puede llegarte una variante tras que esté confirmado, no quieres hacer update entonces

nope

gettransaction te podría devolver la variante no confirmada

exacto, gettransaction te podria devolver una variante no confirmada pues alguna llamada previa a update habria hecho canonica la de menor tamaño

También la confirmada sin querer

en caso que la confirmada no sea la de menor tamaño

no creo, cuando haces una funcion, esa funcion deberia hacer estrictamente lo que dice, RecomputeCanonical debe recomputar la tx canonica, independientemente del estado. Es un helper.
Lo que debes evitar es llamar a RecomputeCanonical si hay una tx confirmada, por eso el !isConfirmed es el condicional para llamar a RecomputeCanonical

Pero como se obentendria esa tx? Por memoool no y la wallet no debería dejarte crear una tx sin utxo

Sisi pero me refería a que una vez confirmado esa casuística es difícil, por qué la tx no se llegaría a ver

mmmm no se si se podria forzar con un broadcasttransaction

Nada, ntr