arturo icon indicating copy to clipboard operation
arturo copied to clipboard

[Values/printable] `codify` could work better for String values

Open github-actions[bot] opened this issue 1 year ago • 0 comments

[Values/printable] codify could work better for String values right now, it also escape Unicode values and that may not be what we need on the other hand, perhaps it shouldn't even be used for that type of things(?)

https://github.com/arturo-lang/arturo/blob/471838c7b3dca5bf2d217fff567c43d69c498ff1/src/vm/values/printable.nim#L509

                    result &= "{\n" & repeat("        ",level+1) & splitl & "\n" & repeat("        ",level) & "}"
                else:
                    # TODO(Values/printable) `codify` could work better for String values
                    #  right now, it also escape Unicode values and that may not be what we need
                    #  on the other hand, perhaps it shouldn't even be used for that type of things(?)
                    #  labels: vm,values,open discussion
                    result &= escape(v.s)
        of Word         : result &= v.s
        of Literal      : result &= "'" & v.s

1a3b2bc1c560d72484d895d54e90bc9744eb7a82

github-actions[bot] avatar Nov 02 '23 16:11 github-actions[bot]