[patch #5672] Yet more procedure / dictionary changes.

[Ben Pfaff]

Follow-up Comment #1, patch #5672 (project pspp):

The change to dict_delete_var worries me: now the callback will be called
with the vardict indexes wrong, because variables have been moved around in
the vars array but their indexes have not been updated in the vardicts.  This
means that some operations on variables will assert-fail (anything that
asserts on dict_contains_var, for example) if called within the callback. 
Seems worrisome given the stated purpose of this change.

Otherwise fine by me.

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?5672>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/