Refactor for clarity: I think this is a good case for goto

parent 73ae4228
...@@ -1611,14 +1611,15 @@ stvfe_mutate(struct worker *wrk, struct objcore *oc) ...@@ -1611,14 +1611,15 @@ stvfe_mutate(struct worker *wrk, struct objcore *oc)
CHECK_OBJ_NOTNULL(oh, OBJHEAD_MAGIC); CHECK_OBJ_NOTNULL(oh, OBJHEAD_MAGIC);
if (oc->refcnt == 1 && !Lck_Trylock(&oh->mtx)) { if (oc->refcnt == 1 && !Lck_Trylock(&oh->mtx)) {
if (oc->refcnt == 1) { if (oc->refcnt != 1)
goto unlock;
stv = oc_stv(wrk, oc); stv = oc_stv(wrk, oc);
stvfe = stv_stvfe(stv); stvfe = stv_stvfe(stv);
if (stv != stvfe->memstv) {
AZ(retval); AZ(retval);
if (stv != stvfe->memstv)
goto unlock; goto unlock;
}
assert_memstv(stvfe, stv); assert_memstv(stvfe, stv);
AN(oc->stobj->priv); AN(oc->stobj->priv);
...@@ -1633,7 +1634,7 @@ stvfe_mutate(struct worker *wrk, struct objcore *oc) ...@@ -1633,7 +1634,7 @@ stvfe_mutate(struct worker *wrk, struct objcore *oc)
stvfe->stats->c_mem_obj_mutate++; stvfe->stats->c_mem_obj_mutate++;
wrk->stats->n_vampireobject++; wrk->stats->n_vampireobject++;
wrk->stats->n_object--; wrk->stats->n_object--;
}
unlock: unlock:
Lck_Unlock(&oh->mtx); Lck_Unlock(&oh->mtx);
} }
......
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