Hello Jerome, Thank you for the contribution. The patch looks fine. I have only some proposals. 1. We don't use bb-10.2-ext any more. Please use 10.3 instead. 2. You mapped "TRIM_ORACLE" to SYM(TRIM). I guess you wanted to avoid grammar duplication for "TRIM" and "TRIM_ORACLE". It's good to avoid grammar duplication. However, this way introduces some problems: - using yyval directly won't work in all cases - "TRIM_ORACLE" becomes a reserved word. It must be non-reserved. So I propose to do the same thing in a different way: - Let's introduce a new rule "trim_operands" and a new structure Lex_trim_st to store TRIM operands. - Let's have a separate token TRIM_ORACLE 3. Create_func_trim_oracle is not really needed. Please review a modified patch attached. Thanks! On 03/15/2018 11:22 AM, jerome brauge wrote:
Hello Alexander,
Can you review this patch for trim functions for Oracle ?
Regards,
Jérôme.