Commit bb0c2acf authored by Yoni Fogel's avatar Yoni Fogel

refs #5153 minor cleanups from review

git-svn-id: file:///svn/toku/tokudb@45006 c7de825b-a66e-492c-adef-691d508d4ae1
parent d0227a5e
...@@ -3328,7 +3328,7 @@ toku_ft_handle_clone(FT_HANDLE *cloned_ft_handle, FT_HANDLE ft_handle, TOKUTXN t ...@@ -3328,7 +3328,7 @@ toku_ft_handle_clone(FT_HANDLE *cloned_ft_handle, FT_HANDLE ft_handle, TOKUTXN t
int r; int r;
FT_HANDLE result_ft_handle; FT_HANDLE result_ft_handle;
r = toku_ft_handle_create(&result_ft_handle); r = toku_ft_handle_create(&result_ft_handle);
invariant_zero(r); resource_assert_zero(r);
// we're cloning, so the handle better have an open ft and open cf // we're cloning, so the handle better have an open ft and open cf
invariant(ft_handle->ft); invariant(ft_handle->ft);
......
...@@ -1284,8 +1284,7 @@ toku_lt_create(toku_lock_tree** ptree, ...@@ -1284,8 +1284,7 @@ toku_lt_create(toku_lock_tree** ptree,
} }
// allocate a tree, initialized to zeroes // allocate a tree, initialized to zeroes
tmp_tree = toku_xmalloc(sizeof(*tmp_tree)); XCALLOC(tmp_tree);
memset(tmp_tree, 0, sizeof(*tmp_tree));
tmp_tree->mgr = mgr; tmp_tree->mgr = mgr;
tmp_tree->compare_fun = compare_fun; tmp_tree->compare_fun = compare_fun;
tmp_tree->lock_escalation_allowed = TRUE; tmp_tree->lock_escalation_allowed = TRUE;
...@@ -1358,9 +1357,9 @@ toku_ltm_get_lt(toku_ltm* mgr, toku_lock_tree** ptree, DICTIONARY_ID dict_id, DE ...@@ -1358,9 +1357,9 @@ toku_ltm_get_lt(toku_ltm* mgr, toku_lock_tree** ptree, DICTIONARY_ID dict_id, DE
// mutually null or mutually non-null. // mutually null or mutually non-null.
if ((on_close_callback == NULL) != (on_close_callback == NULL)) { if ((on_close_callback == NULL) != (on_close_callback == NULL)) {
r = EINVAL; r = EINVAL;
goto out; goto cleanup;
} }
ltm_mutex_lock(mgr); ltm_mutex_lock(mgr);
map = toku_idlth_find(mgr->idlth, dict_id); map = toku_idlth_find(mgr->idlth, dict_id);
if (map != NULL) { if (map != NULL) {
...@@ -1384,10 +1383,10 @@ toku_ltm_get_lt(toku_ltm* mgr, toku_lock_tree** ptree, DICTIONARY_ID dict_id, DE ...@@ -1384,10 +1383,10 @@ toku_ltm_get_lt(toku_ltm* mgr, toku_lock_tree** ptree, DICTIONARY_ID dict_id, DE
if (on_create_callback) { if (on_create_callback) {
on_create_callback(tree, on_create_extra); on_create_callback(tree, on_create_extra);
} else { } else {
assert(on_close_callback == NULL); invariant(on_close_callback == NULL);
} }
tree->on_close_callback = on_close_callback; tree->on_close_callback = on_close_callback;
lt_set_dict_id(tree, dict_id); lt_set_dict_id(tree, dict_id);
/* add tree to ltm */ /* add tree to ltm */
r = ltm_add_lt(mgr, tree); r = ltm_add_lt(mgr, tree);
...@@ -1405,12 +1404,12 @@ toku_ltm_get_lt(toku_ltm* mgr, toku_lock_tree** ptree, DICTIONARY_ID dict_id, DE ...@@ -1405,12 +1404,12 @@ toku_ltm_get_lt(toku_ltm* mgr, toku_lock_tree** ptree, DICTIONARY_ID dict_id, DE
goto cleanup; goto cleanup;
} }
added_to_idlth = TRUE; added_to_idlth = TRUE;
map = toku_idlth_find(mgr->idlth, dict_id); map = toku_idlth_find(mgr->idlth, dict_id);
assert(map); assert(map);
map->tree = tree; map->tree = tree;
/* No add ref needed because ref count was set to 1 in toku_lt_create */ /* No add ref needed because ref count was set to 1 in toku_lt_create */
*ptree = tree; *ptree = tree;
r = 0; r = 0;
cleanup: cleanup:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment