[Maria-developers] Please review a 10.3 patch "Moving functions cast_stmt_xxx and add_select_to_union_list as methods to LEX"
![](https://secure.gravatar.com/avatar/68ff00c1752bf8aede68a05f77ebd28b.jpg?s=120&d=mm&r=g)
Hello Sanja, As we agreed during our discussion on slack, we should port a few preparatory patches from bb-10.2-compatibility to 10.3, to make our further compatibility related development easier. This is the first patch. Please review. Thanks!
![](https://secure.gravatar.com/avatar/548f6a7ca82fc307ee146f412b04905c.jpg?s=120&d=mm&r=g)
Hi, Alexander! On 11/24/2016 07:09 AM, Alexander Barkov wrote:
Hello Sanja,
As we agreed during our discussion on slack, we should port a few preparatory patches from bb-10.2-compatibility to 10.3, to make our further compatibility related development easier.
This is the first patch.
Please review.
Thank you , the patch is OK to push. I a bit dislike this mix Lex/lex thing, but we can not do something about it, logic is clear - one use is Lex several use cases then lex=Lex. [skip]
![](https://secure.gravatar.com/avatar/39b623a1559cf9c69ac3d9d4fb44e7fe.jpg?s=120&d=mm&r=g)
Hi, Oleksandr! On Nov 24, Oleksandr Byelkin wrote:
Thank you , the patch is OK to push. I a bit dislike this mix Lex/lex thing, but we can not do something about it, logic is clear - one use is Lex several use cases then lex=Lex.
Not really. Lex is YYTHD->lex. And many years ago YYTHD was current_thd. This LEX *lex=Lex; mantra was used to avoid calling current_thd too often. Now thd is passed as an argument to yyparse, so Lex no longer uses current_thd and there's no need to "optimize" it into a local variable. Old code still does it, though. Regards, Sergei Chief Architect MariaDB and security@mariadb.org
participants (3)
-
Alexander Barkov
-
Oleksandr Byelkin
-
Sergei Golubchik