John Darrington <j...@darrington.wattle.id.au> writes: > The attached patches I believe represent improvements to the psppsheet branch > They don't fix all of the problems I had hoped, but I think it'll make the > branch easier to work on. > > A review would be appreciated.
I have a few questions. Do you have a rule of thumb that explains what should go in destroy, dispose, and finalize? The difference between destroy and dispose seems especially difficult to grasp. I wonder whether some of these changes to the destroy/dispose division are correct, particularly in the case of PsppSheetView since there the code comes straight from GtkTreeView with few changes, but I don't know how to decide that from "first principles". Some of the commits adjust refcounting of the UI manager. The commit messages mention that this fixes problems but don't explain further. Can you say more? The patch "Reduce the flicker when redrawing the toolbar and menubar." appears to leak a reference in psppire_data_editor_split_window(). At least, I see nothing that will eventually unref de->old_vbox_widget. Thanks, Ben. P.S. "git am" told me that some of the patches add trailing whitespace, not that that's really a big deal. _______________________________________________ pspp-dev mailing list pspp-dev@gnu.org https://lists.gnu.org/mailman/listinfo/pspp-dev
