HomePhabricator

Reorganize(): remove spurious TxnAbort()
b52a27053850Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Reorganize(): remove spurious TxnAbort()

If Reorganize() fails, then its caller, CBlock::SetBestChain(),
will call TxnAbort().

Redundant TxnAbort() calls are harmless. The second will return an
error return value, with no other side effects. TxnAbort() return
values are generally never checked. The impact is nil.

Details

Provenance
Jeff Garzik <jgarzik@exmulti.com>Authored on May 14 2012, 16:21
Jeff Garzik <jgarzik@redhat.com>Committed on May 20 2012, 00:40
schancelPushed on Jan 5 2018, 21:58
schancelPushed on Jan 5 2018, 21:39
schancelPushed on Jan 5 2018, 21:17
Parents
rSTAGINGcde87ee75cd9: Merge pull request #1360 from jgarzik/datasync
Branches
Unknown
Tags
Unknown

Event Timeline

Jeff Garzik <jgarzik@redhat.com> committed rSTAGINGb52a27053850: Reorganize(): remove spurious TxnAbort() (authored by Jeff Garzik <jgarzik@exmulti.com>).May 20 2012, 00:40