Merge lp:~laurynas-biveinis/percona-server/bug1262439-5.1 into lp:percona-server/5.1

Proposed by Laurynas Biveinis
Status: Merged
Approved by: Alexey Kopytov
Approved revision: no longer in the source branch.
Merged at revision: 611
Proposed branch: lp:~laurynas-biveinis/percona-server/bug1262439-5.1
Merge into: lp:percona-server/5.1
Diff against target: 41 lines (+5/-5)
1 file modified
Percona-Server/sql/sql_yacc.yy (+5/-5)
To merge this branch: bzr merge lp:~laurynas-biveinis/percona-server/bug1262439-5.1
Reviewer Review Type Date Requested Status
Alexey Kopytov (community) Approve
Review via email: mp+216886@code.launchpad.net

Description of the change

Fix compilation with Bison 3.

Applied the smaller patch and not the MariaDB one. While the latter is more thorough, the smaller one is enough and I expect to replace any patch with Oracle one at the next point version merge. For that a smaller patch will be easier to revert.

http://jenkins.percona.com/job/percona-server-5.1-param/604/ and tested under Ubuntu 14.04 locally.

To post a comment you must log in.
Revision history for this message
Laurynas Biveinis (laurynas-biveinis) wrote :
Revision history for this message
Alexey Kopytov (akopytov) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'Percona-Server/sql/sql_yacc.yy'
--- Percona-Server/sql/sql_yacc.yy 2013-06-03 03:53:55 +0000
+++ Percona-Server/sql/sql_yacc.yy 2014-04-23 13:48:41 +0000
@@ -27,8 +27,6 @@
27** The type will be void*, so it must be cast to (THD*) when used.27** The type will be void*, so it must be cast to (THD*) when used.
28** Use the YYTHD macro for this.28** Use the YYTHD macro for this.
29*/29*/
30#define YYPARSE_PARAM yythd
31#define YYLEX_PARAM yythd
32#define YYTHD ((THD *)yythd)30#define YYTHD ((THD *)yythd)
33#define YYLIP (& YYTHD->m_parser_state->m_lip)31#define YYLIP (& YYTHD->m_parser_state->m_lip)
3432
@@ -64,7 +62,7 @@
64 ulong val= *(F); \62 ulong val= *(F); \
65 if (my_yyoverflow((B), (D), &val)) \63 if (my_yyoverflow((B), (D), &val)) \
66 { \64 { \
67 yyerror((char*) (A)); \65 yyerror(yythd, (char*) (A)); \
68 return 2; \66 return 2; \
69 } \67 } \
70 else \68 else \
@@ -159,7 +157,7 @@
159 to abort from the parser.157 to abort from the parser.
160*/158*/
161159
162void MYSQLerror(const char *s)160void MYSQLerror(void *yythd, const char *s)
163{161{
164 THD *thd= current_thd;162 THD *thd= current_thd;
165163
@@ -675,7 +673,9 @@
675bool my_yyoverflow(short **a, YYSTYPE **b, ulong *yystacksize);673bool my_yyoverflow(short **a, YYSTYPE **b, ulong *yystacksize);
676%}674%}
677675
678%pure_parser /* We have threads */676%pure-parser /* We have threads */
677%parse-param { void *yythd }
678%lex-param { void *yythd }
679/*679/*
680 Currently there are 169 shift/reduce conflicts.680 Currently there are 169 shift/reduce conflicts.
681 We should not introduce new conflicts any more.681 We should not introduce new conflicts any more.

Subscribers

People subscribed via source and target branches