On 11/12/2014 02:29, Marvin Humphrey wrote:
IIRC `Err_error` was the original location but its functionality was moved
into a Perl scalar so that the variable would be thread-local under Perl
ithreads.
I think we should avoid symbolic replacements which include an article like
"the" -- too inflexible, too much mental gymnastics to use in context.
Instead, I suggest we should map @error to a proper name, but keep it hidden
-- i.e. "current_error" in the C bindings would be renamed to "Err_error" but
would retain its `static` qualifier.
I agree with using the pseudo-symbol `[](cfish:@error)`, because we will be
special-casing the translation. Prepending `@` seems like a decent choice.
Maybe we should simply reword the documentation from
...sets Err_error when...
to
...sets the global [](cfish:cfish.Err) object when...
or
...sets the global [](cfish:cfish.Err) object returned by
[](cfish:cfish.Err.get_error) when...
and use "Clownfish->error" instead of "get_error" for the Perl documentation.
So we wouldn't need the `@error`symbol at all.
Nick
