Why are there multiple `InternalOutputSubstitutionError::OutputSubstitutionDisabled(& str)` variants?

[DanGould]

I'm confused why there are multiple OutputSubstitutionDisabled(&str) variants instead of two sub-variants that don't take string context.

I see:

1

InternalOutputSubstitutionError::OutputSubstitutionDisabled(
                                    "Decreasing the receiver output value is not allowed",
                                )

2

InternalOutputSubstitutionError::OutputSubstitutionDisabled(
                                    "Changing the receiver output script pubkey is not allowed",
                                )

Of which the one appropriate error would be better named

InternalOutputSubstitutionError::Disabled

Originally posted by @DanGould in #546 (comment)

[spacebear21]

Having two sub-variants instead of string context makes sense imo.

[benalleng]

I can go ahead and make a new sub-variant for these if you don't plan on including a fix in #546

[DanGould]

yes please @benalleng