src/data/procedure.c:808

[John Darrington]

Isn't the right hand side of this statement redundant ?

 ok = trns_chain_destroy (ds->temporary_trns_chain) && ok;

Also, I wonder about the next two lines: 
 
  ds->permanent_trns_chain = ds->cur_trns_chain = trns_chain_create ();
  ds->temporary_trns_chain = NULL;

Why assign ds->permanent_trns_chain to something, and then to
immediately to something else?

J'

-- 
PGP Public key ID: 1024D/2DE827B3 
fingerprint = 8797 A26D 0854 2EAB 0285  A290 8A67 719C 2DE8 27B3
See http://pgp.mit.edu or any PGP keyserver for public key.

pgpuRm22pkJ_q.pgp
Description: PGP signature