On 2019-12-08 17:10, Mark Rotteveel wrote:
For consistency of the grammar, I think it is better if the NATIVE
option is also prefixed with TO,
So instead of SET BIND <type> NATIVE, I think it should be SET BIND
<type> TO NATIVE. This would match with the form of SET BIND <type> TO
LEGACY, and grammatically, this also makes more sense to me, because
you instruct Firebird to bind the type *to* a type (even if you are
just saying to bind it to itself).
If others agree - no problems, that's 5 minutes change.
As a separate issue, I noticed a discrepancy: DECFLOAT is an alias for
DECFLOAT(34), but SET BIND DECFLOAT TO VARCHAR will map both
DECFLOAT(16) and DECFLOAT(34), while SET BIND DECFLOAT(34) TO VARCHAR
will only map DECFLOAT(34). Looking at the parser rules, I can't see
how it arrives at a different result for this.
That's as designed (see docs from previous email). Processing is done
not in grammar parser - therefire you do not see it. Look at
jrd/Coercion.cpp if you are interested how it works.
Firebird-Devel mailing list, web interface at
https://lists.sourceforge.net/lists/listinfo/firebird-devel
