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).

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.

Mark
--
Mark Rotteveel


Firebird-Devel mailing list, web interface at 
https://lists.sourceforge.net/lists/listinfo/firebird-devel

Reply via email to