Re: [hackers] [sbase][PATCH] du: Dedup hardlinks

From: Roberto E. Vargas Caballero <k0ga_AT_shike2.net>
Date: Thu, 20 Mar 2025 10:20:30 +0100

Hi,

Quoth Roberto E. Vargas Caballero <k0ga_AT_shike2.net>:
> Quoth remph <lhr_AT_disroot.org>:
> > +struct device {
> > + dev_t devno;
> > + void * inodetree;
> > +};
> > +
>
> I can see that you define a device struct, that contains a inode
> table, but I think it is just easier to create a struct like:
>
> struct file {
> dev_t dev;
> ino_t inode;
> };
>
>
> and adjust the compare function to use both fields. Am I missing
> something?

I reworked it to use only one tree. Please, tell me what you think
about this other version (some small changes were also done, like
for example changing isduplicated to duplicated, because the is[a-z]*
namespace is reserved to the implementation, and some bracing changes
to accomodate to the style used in suckless).

--- 8< ---
Received on Thu Mar 20 2025 - 10:20:30 CET