From: Karthik Nayak Date: Wed, 07 Aug 2024 08:03:00 GMT Subject: Re: [GSoC][PATCH v15 3/9] fsck: make "fsck_error" callback generic Message-ID: In-Reply-To: shejialuo writes: > The "fsck_error" callback is designed to report the objects-related > error messages. It accepts two parameter "oid" and "object_type" which > is not generic. In order to provide a unified callback which can report > either objects or refs, remove the objects-related parameters and add > the generic parameter "void *fsck_report". > > Create a new "fsck_object_report" structure which incorporates the > removed parameters "oid" and "object_type". Then change the > corresponding references to adapt to new "fsck_error" callback. > So since my last review, we've now changed the function to accept a 'void *' parameter, this makes sense, this way the refs error function and the objects error function can now only receive the data they care about. [snip]