Hi.

At the end of test_if_skip_sort_order() you put (in 5.3):

17959           /*
17960             orig_cond is a part of pre_idx_push_cond,
17961             no need to restore it.
17962           */
17963           orig_cond= 0;
17964           orig_cond_saved= false;
17965   if (orig_cond_saved && !changed_key)
17966     tab->set_cond(orig_cond);

Besides incorrect indentation, you set orig_cond_saved to false, and in
the next line writes an if() statement that can only work if
orig_cond_saved is true. If orig_cond_saved=false is correct, then this
if() can be completely removed. Or, may be, orig_cond_saved should not
be set to false?

Regards,
Sergei

_______________________________________________
Mailing list: https://launchpad.net/~maria-developers
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~maria-developers
More help   : https://help.launchpad.net/ListHelp

Reply via email to