Code review comment for lp:~oif-team/geis/geis2_add_regions

Revision history for this message
Chase Douglas (chasedouglas) wrote :

I know this has been merged, but I would like to see geis_region_delete() renamed to something like geis_region_deref() or geis_region_release(). Delete implies that the object will be freed immediately, which is not what happens here if a reference is held elsewhere.

« Back to merge proposal