Merge lp:~vkolesnikov/pbxt/pbxt-20-build-problem-5131 into lp:pbxt/2.0

Proposed by Vladimir Kolesnikov
Status: Merged
Merged at revision: 925
Proposed branch: lp:~vkolesnikov/pbxt/pbxt-20-build-problem-5131
Merge into: lp:pbxt/2.0
Diff against target: 23 lines (+5/-1)
1 file modified
src/discover_xt.cc (+5/-1)
To merge this branch: bzr merge lp:~vkolesnikov/pbxt/pbxt-20-build-problem-5131
Reviewer Review Type Date Requested Status
PBXT Core Pending
Review via email: mp+42091@code.launchpad.net

Description of the change

I did not get any feedback from the original reporter so far, but I believe this fix will be useful anyway. The problem with build was that jumps to some labels bypassed local variable declarations.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/discover_xt.cc'
--- src/discover_xt.cc 2010-06-15 08:48:11 +0000
+++ src/discover_xt.cc 2010-11-29 11:37:04 +0000
@@ -1302,6 +1302,10 @@
13021302
1303 mylex.create_info.frm_only = 1;1303 mylex.create_info.frm_only = 1;
1304 mylex.create_info.default_table_charset = system_charset_info;1304 mylex.create_info.default_table_charset = system_charset_info;
1305
1306#if MYSQL_VERSION_ID > 50130
1307 partition_info *part_info= NULL;
1308#endif
1305 1309
1306 /* setup the column info. */1310 /* setup the column info. */
1307 while (info->field_name) { 1311 while (info->field_name) {
@@ -1350,7 +1354,7 @@
1350 1354
1351 /* Create an internal temp table */1355 /* Create an internal temp table */
1352#if MYSQL_VERSION_ID > 501301356#if MYSQL_VERSION_ID > 50130
1353 partition_info *part_info= thd->work_part_info;1357 part_info= thd->work_part_info;
1354 1358
1355 thd->work_part_info = NULL;1359 thd->work_part_info = NULL;
1356#endif1360#endif

Subscribers

People subscribed via source and target branches