Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> writes: > - .for_each_reflog_ent = packed_for_each_reflog_ent, > - .for_each_reflog_ent_reverse = packed_for_each_reflog_ent_reverse, > - .reflog_exists = packed_reflog_exists, > - .create_reflog = packed_create_reflog, > - .delete_reflog = packed_delete_reflog, > - .reflog_expire = packed_reflog_expire > + .for_each_reflog_ent = NULL, > + .for_each_reflog_ent_reverse = NULL, > + .reflog_exists = NULL, > + .create_reflog = NULL, > + .delete_reflog = NULL, > + .reflog_expire = NULL, Yup, the entry in vtable being set to NULL is a more direct sign the readers would appreciate than a name natural for the method, whose definition needs to be consulted before the readers can realize that it is a "missing" method. Very sensible. Thanks.
