sca@stripped a écrit, Le 05.07.2010 04:18:
> I'd like to offer the following fix for BUG#49129 under SCA:
Thank you very much.
> @@ -3116,7 +3130,12 @@
> JOIN_TAB *j, *jump_to= tab-1;
> for (j= tab; j != tab + pos->n_sj_tables; j++)
> - if (!tab->emb_sj_nest)
> + /*
> + NOTE: this loop probably doesn't do the right thing for the case
> + where FirstMatch's duplicate-generating range is interleaved with
> + "unrelated" tables (as specified in WL#3750, section 2.2).
> + */
> + if (!j->emb_sj_nest)
> jump_to= tab;
The Sun guys had come through this oddity too; the latest code in their
tree has this comment:
jump_to= tab; /// @todo fix this (BUG#51457)
(the for() loop was moved elsewhere but it's irrelevant here). In
BUG#51457 there are examples of wrong results generated by this code.