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.


Attachment: pgpg977Fg7Zkh.pgp
Description: PGP signature

_______________________________________________
pspp-dev mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/pspp-dev

Reply via email to