search

dgpv / bsst

B'SST: Bitcoin-like Script Symbolic Tracer
Other
34 stars 5 forks source link

Maybe use special chars for 'auto-added' data identifiers #5

Closed dgpv closed 1 year ago

dgpv commented 1 year ago

Now we just disallow user-provided data identifiers that start with wit\d+ (regex) or tx_ , to prevent conflict with automatically added identifiers for witnesses and transaction fields.

Maybe we should use some special char to make sure name clash is not possible

But I did not find the char that would seem good. Need to not degrade readability of the report.

~wit0~, ~tx_num_inputs~ maybe ? or *wit32*, *tx_nVersion* ?

dgpv commented 1 year ago

I decided to rename 'data identifiers' to 'data references' and prpend & to them in the report, so that they be visually different from the auto-added references like witnesses and transaction fields.

Done in https://github.com/dgpv/bsst/commit/688a312fef922f2c7602648ea80cb4a898bed5ac