lambdaclass/cairo-vm

perf: plain and starknet etc use pub const string

jujube opened this issue · 1 comments

_name: String::from("plain"),

change to format
pub const OUTPUT_BUILTIN_NAME: &str = "output_builtin";

Literals are functionally the same. In both cases, as long as the type remains String, you would be allocating a buffer and copying the string from the data segment or similar (depends on the platform). Some may consider one is cleaner, but other than that it is unlikely to have any observable difference.