Fix Blob_t vs ContactBlob type differences

[dfandrich]

These structs are defined differently with a potentially different
memory layouts so they cannot be treated as the same.

This difference causes a compile failure with gcc 15.1

[dfandrich]

I should mention that I've only compiled this patch, but I haven't tested it (though it's quite simple to inspect).

[eklausme]

Your proposed change is valid and good.

For the Arch Linux package jpilot I simply used a sed and a cast to fix the glitch during build time:

sed 's/return Contact_add_blob(c, blob);/return Contact_add_blob(c, (void*)blob);/' jp-contact.c > /tmp/jp-contact.c

The problem you addressed here is a permanent nuisance between the different versions of pilot-link.

1. According the latest master branch for pilot-link, we have pi-contact.h:

extern int Contact_add_blob
    PI_ARGS((struct Contact *, Blob_t *));

2. According the source code contact.c in pilot-link, it is

int Contact_add_blob(struct Contact *c, Blob_t *blob) { ... }

3. On Arch Linux, however, pilot-link's pi-contact.h says

extern int Contact_add_blob
    PI_ARGS((struct Contact *, struct ContactBlob *));

I assume your distros uses the same mess.

So, for me it looks that we are fixing something in J-Pilot that should be fixed in pilot-link's repositories for the various distros.

[dfandrich]

My distro, as well as https://github.com/jichu4n/pilot-link/, both have struct ContactBlob *blob as the parameter in both contact.c and pi-contact.h. Did someone patch the version in your distro? I don't see how such a mismatch between prototype and function would work given that the types are of different sizes (at least on some platforms). gcc's latest release is even more picky than normal about that kind of thing. Unless there's a new upstream that I haven't seen that has changed those types, I think jpilot should be fixed to match pilot-link.

[eklausme]

"Your" repository from jichu4n is 9 years old.

Please have a look at the history of pi-contact.h here: commit history.

I consider desrod's repository to be the "official" pilot-link repository. However, not everyone agrees with that.